Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore(gatsby-core-utils): cpu-core-count to TS #22048

Conversation

danielkov
Copy link
Contributor

Collection ticket for these tasks: #22041
☂ ticket: #21995

@@ -48,4 +47,4 @@ const cpuCoreCount = ignoreEnvVar => {
}
}

module.exports = cpuCoreCount
export default cpuCoreCount
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
export default cpuCoreCount
export { cpuCoreCount }

And adjust import in https://github.com/gatsbyjs/gatsby/blob/master/packages/gatsby-core-utils/src/index.js

@blainekasten blainekasten changed the base branch from master to ts/gatsby-core-utils March 9, 2020 19:56
@blainekasten blainekasten merged commit 25bf063 into gatsbyjs:ts/gatsby-core-utils Mar 9, 2020
blainekasten added a commit that referenced this pull request Mar 10, 2020
* chore(gatsby-core-utils): get-config-store to TS (#22051)

* chore(gatsby-core-utils): get-config-store to TS

* chore(gatsby-core-utils): change getConfigStore to named export

* chore(gatsby-core-utils): ci to TS (#22047)

* chore(gatsby-core-utils): package.json added type

* chore(gatsby-core-utils): mocks convert ci-info to TS

* chore(gatsby-core-utils): ci to TS

* chore(gatsby-core-utils): add @types/ci-info

* chore(gatsby-core-utils): re-install dependencies to fix order

* chore(gatsby-core-utils): cpu-core-count to TS (#22048)

* chore(gatsby-core-utils): create-content-digest to TS (#22049)

* chore(gatsby-core-utils): tests create-content-digest to TS

* chore(gatsby-core-utils): create-content-digest to TS

* chore(gatsby-core-utils): create-require-from-path to TS (#22050)

* chore(gatsby-core-utils): get-gatsby-version to TS (#22052)

* chore(gatsby-core-utils): index to TS (#22053)

* chore(gatsby-core-utils): index to TS

* chore(gatby-core-utils): changed getConfigStore to named export in #22051

* chore(gatsby-core-utils): path to ts (#22054)

* chore(gatsby-core-utils): tests path to TS

* chore(gatsby-core-utils): path to ts

* chore(gatsby-core-utils): url to TS (#22056)

* chore(gatsby-core-utils): tests url to TS

* chore(gatsby-core-utils): url to TS

* migrate more files

* fix the rest

* better exports

* fix lints

* fix building

* fix test

Co-authored-by: Daniel Emod Kovacs <kovacsemod@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants