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

Support --use-version argument with shipit #2075

Merged
merged 6 commits into from Sep 15, 2021
Merged

Conversation

kelyvin
Copy link
Contributor

@kelyvin kelyvin commented Sep 14, 2021

What Changed

Updated the auto shipit command to enable setting the --use-version flag. (Currently only supported for the npm plugin)

Additional Changes:

  • Include .editorconfig to force consistency across dev environments
  • Include a function getLatestReleasedTagWithPattern to provide better support for GitHub "release" version based on a provided pattern.

Why

There are use cases where we want to specify the version to release through shipit. In particular, when maintaining multiple old version branches, version calculations are computed incorrectly. In order to support complex semantic release workflows through npm, this is a temporary solution until the root cause can be determined (i.e. oldRelease()) .

Todo:

  • Add tests
  • Add docs

Change Type

Indicate the type of change your pull request is:

  • documentation
  • patch
  • minor
  • major

@adierkens adierkens added the patch Increment the patch version when merged label Sep 14, 2021
@codecov
Copy link

codecov bot commented Sep 14, 2021

Codecov Report

Merging #2075 (41063f6) into main (40f5c49) will decrease coverage by 0.05%.
The diff coverage is 60.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #2075      +/-   ##
==========================================
- Coverage   80.24%   80.19%   -0.06%     
==========================================
  Files          66       66              
  Lines        5397     5403       +6     
  Branches     1251     1254       +3     
==========================================
+ Hits         4331     4333       +2     
- Misses        706      709       +3     
- Partials      360      361       +1     
Impacted Files Coverage Δ
plugins/npm/src/index.ts 72.87% <42.85%> (-0.26%) ⬇️
packages/core/src/auto.ts 77.56% <71.42%> (-0.17%) ⬇️
packages/cli/src/parse-args.ts 97.50% <100.00%> (+0.06%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 0b7399f...41063f6. Read the comment docs.

Copy link
Collaborator

@hipstersmoothie hipstersmoothie left a comment

Choose a reason for hiding this comment

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

Looking good. just a few things I need addressed

.editorconfig Outdated Show resolved Hide resolved
packages/core/src/git.ts Outdated Show resolved Hide resolved
@kelyvin
Copy link
Contributor Author

kelyvin commented Sep 15, 2021

@hipstersmoothie I have addressed your PR comments, let me know if you need anything else!

@hipstersmoothie hipstersmoothie added minor Increment the minor version when merged and removed patch Increment the patch version when merged labels Sep 15, 2021
@hipstersmoothie hipstersmoothie merged commit 9f1a57d into intuit:main Sep 15, 2021
@adierkens
Copy link
Collaborator

🚀 PR was released in v10.32.0 🚀

@adierkens adierkens added the released This issue/pull request has been released. label Sep 15, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
minor Increment the minor version when merged released This issue/pull request has been released.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants