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

annotate files relative to git root, instead of cwd, so that it works in nested packages in monorepos #151

Merged

Conversation

PaulGiletich
Copy link
Contributor

@PaulGiletich PaulGiletich commented Mar 2, 2023

probably a major change?
although very unlikely, but still can break somebody.

confirmed working in our monorepo

used execa v1 instead of the latest one because dependencies already use it. just to dedupe versions

@PaulGiletich PaulGiletich changed the title annotate files relative to git root, not to cwd annotate files relative to git root, instead of cwd, so that it works in nested packages in monorepos Mar 2, 2023
@PaulGiletich PaulGiletich marked this pull request as ready for review March 2, 2023 02:03
@PaulGiletich
Copy link
Contributor Author

@hipstersmoothie sorry to bother, but seeing as hipstersmoothie/eslint-formatter-github#114 is merged, i though maybe you could please review this one as well when you have a minute?

@hipstersmoothie hipstersmoothie added the patch Increment the patch version when merged label Mar 2, 2023
@hipstersmoothie
Copy link
Owner

@PaulGiletich fix merge conflicts and I'll get this merged!

@PaulGiletich
Copy link
Contributor Author

@hipstersmoothie done!

@hipstersmoothie hipstersmoothie merged commit ba8afbe into hipstersmoothie:master Mar 2, 2023
@hipstersmoothie
Copy link
Owner

🚀 PR was released in v1.1.1 🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
patch Increment the patch version when merged released
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants