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: upgrade conventional commits #640

Closed
wants to merge 4 commits into from

Conversation

@travi
Copy link
Member

travi commented May 7, 2024

Please make sure our tests are passing. npm test should be enough to verify this locally.

package.json Outdated
Comment on lines 28 to 33
"conventional-changelog-atom": "^5.0.0",
"conventional-changelog-conventionalcommits": "^8.0.0",
"conventional-changelog-ember": "^5.0.0",
"conventional-changelog-eslint": "^6.0.0",
"conventional-changelog-express": "^5.0.0",
"conventional-changelog-jshint": "^5.0.0",
Copy link
Member

Choose a reason for hiding this comment

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

Please keep dev dependencies pinned to specific versions

@travi
Copy link
Member

travi commented May 10, 2024

Please make sure our tests are passing. npm test should be enough to verify this locally.

@icco the tests continue to fail after your follow up to pin the dev dependencies. are you interested in working through the breaking changes related to these updates in order to help us move this forward?

@icco
Copy link
Author

icco commented May 10, 2024

Please make sure our tests are passing. npm test should be enough to verify this locally.

@icco the tests continue to fail after your follow up to pin the dev dependencies. are you interested in working through the breaking changes related to these updates in order to help us move this forward?

Yeah been trying to get the tests pass locally with no luck. I believe there's an underlying bug in the import plugin you use but can not figure it out.

@travi
Copy link
Member

travi commented May 10, 2024

appreciate the update and the investigation. we are starting to investigate as well, but unsure how much time we can dedicate soon. please feel free to keep us up to date if you make progress, even if that means pushing wip changes that dont get to a point of getting tests to a passing state.

it could also be helpful to start with running the integration tests (npm run test:integration) first for a larger scope feedback loop before getting distracted by the unit tests (npm run test:unit) if the interface changes are unclear.

@travi
Copy link
Member

travi commented May 17, 2024

appreciate the investment here, but closing in favor of #643. we could still use help updating the commit-analyzer plugin in similar ways in case you are still interested and the other PR helps with any complexities that you encountered with this effort

@travi travi closed this May 17, 2024
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.

None yet

3 participants