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: update some major versions #1800

Merged
merged 7 commits into from
Nov 7, 2023
Merged

chore: update some major versions #1800

merged 7 commits into from
Nov 7, 2023

Conversation

ahuth
Copy link
Contributor

@ahuth ahuth commented Nov 6, 2023

Update some major versions of packages.

These are the "easy" ones, that are dropping support for Node versions or other things that don't really affect our code.

@ahuth ahuth requested review from booc0mtaco and a team November 6, 2023 22:23
@ahuth ahuth changed the base branch from main to next November 6, 2023 22:24
@@ -12,7 +12,7 @@ export interface Props {
/**
* BETA: This component is still a work in progress and is subject to change.
*
* `import {{inBraces name}} from "@chanzuckerberg/eds";`
* `import {{inBraces (pascalCase name)}} from "@chanzuckerberg/eds";`
Copy link
Contributor Author

@ahuth ahuth Nov 6, 2023

Choose a reason for hiding this comment

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

I removed the dep on pascal-case rather than update it.

Here I've removed pascal-case from our custom inBraces helper, and am using handlebars's built-in helper instead.

@@ -8,7 +7,7 @@ module.exports = (plop) => {

// This helper allows us to place a variable inside curly braces without spaces
// between the text and the braces.
plop.setHelper('inBraces', (txt) => `{${pascalCase.pascalCase(txt)}}`);
plop.setHelper('inBraces', (txt) => `{${txt}}`);
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Removed pascal-case.

@@ -1,5 +1,4 @@
const pascalCase = require('pascal-case');
const { snakeCase } = require('snake-case');
const { snakeCase } = require('lodash');
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Using lodash for snake case, so we don't have to update snake-case.

path: 'src/components/{{pascalCase name}}/{{pascalCase name}}.stories.tsx',
templateFile: 'plop-templates/Component/Component.stories.tsx.hbs',
path: 'src/components/{{pascalCase name}}/{{pascalCase name}}.stories.ts',
templateFile: 'plop-templates/Component/Component.stories.ts.hbs',
Copy link
Contributor Author

@ahuth ahuth Nov 6, 2023

Choose a reason for hiding this comment

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

yarn plop wasn't working. Needed to update the path here.

It's possible no one is using plop. Should we remove it?

Copy link

codecov bot commented Nov 6, 2023

Codecov Report

Merging #1800 (e300cc5) into next (76c8f4b) will increase coverage by 0.03%.
Report is 1 commits behind head on next.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             next    #1800      +/-   ##
==========================================
+ Coverage   92.57%   92.61%   +0.03%     
==========================================
  Files         147      147              
  Lines        2735     2735              
  Branches      712      712              
==========================================
+ Hits         2532     2533       +1     
  Misses        187      187              
+ Partials       16       15       -1     

see 1 file with indirect coverage changes

📣 Codecov offers a browser extension for seamless coverage viewing on GitHub. Try it in Chrome or Firefox today!

Copy link

github-actions bot commented Nov 6, 2023

size-limit report 📦

Path Size
components 97.81 KB (0%)
styles 34.33 KB (0%)

@ahuth
Copy link
Contributor Author

ahuth commented Nov 7, 2023

Thanks 🙏

@ahuth ahuth merged commit ce79e02 into next Nov 7, 2023
6 checks passed
@ahuth ahuth deleted the ah-majors branch November 7, 2023 18:35
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.

1 participant