8000 feat(bump): option to append skip-ci pattern for bump commit by Wassim-AB · Pull Request #274 · cocogitto/cocogitto · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

feat(bump): option to append skip-ci pattern for bump commit #274

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

Merged
merged 1 commit into from
Aug 1, 2023

Conversation

Wassim-AB
Copy link

No description provided.

@Wassim-AB Wassim-AB changed the title feat: skip-ci option to specify skip-ci pattern for bump commit feat: option to append skip-ci pattern for bump commit Mar 24, 2023
@codecov
Copy link
codecov bot commented Mar 25, 2023

Codecov Report

Merging #274 (6c86f29) into main (a137fcc) will increase coverage by 0.01%.
The diff coverage is 100.00%.

❗ Current head 6c86f29 differs from pull request most recent head fea0ece. Consider uploading reports for the commit fea0ece to get more accurate results

@@            Coverage Diff             @@
##             main     #274      +/-   ##
==========================================
+ Coverage   85.35%   85.36%   +0.01%     
==========================================
  Files          45       45              
  Lines        6239     6239              
==========================================
+ Hits         5325     5326       +1     
+ Misses        914      913       -1     
Impacted Files Coverage Δ
src/command/bump/monorepo.rs 93.19% <100.00%> (+0.35%) ⬆️
src/command/bump/package.rs 92.50% <100.00%> (+0.71%) ⬆️
src/command/bump/standard.rs 97.14% <100.00%> (+0.21%) ⬆️
src/settings/mod.rs 77.82% <100.00%> (+0.09%) ⬆️

... and 3 files with indirect coverage changes

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@Wassim-AB Wassim-AB changed the title feat: option to append skip-ci pattern for bump commit feat(bump): option to append skip-ci pattern for bump commit Mar 31, 2023
@Wassim-AB Wassim-AB force-pushed the feat/skip-ci branch 3 times, most recently from c04aa81 to d1c600e Compare June 13, 2023 14:50
@Wassim-AB
Copy link
Author

Hey @oknozor, I think your merged main into my branch by mistake rather than rebasing it :)

@oknozor
Copy link
Collaborator
oknozor commented Jun 21, 2023

Hey @oknozor, I think your merged main into my branch by mistake rather than rebasing it :)

Absolutely my mistake, sorry. Can you rebase locally and force push ?

@Wassim-AB Wassim-AB force-pushed the feat/skip-ci branch 2 times, most recently from f06f605 to 042dca5 Compare June 21, 2023 09:01
@Wassim-AB
Copy link
Author
Wassim-AB commented Jun 21, 2023

@oknozor No problem, done ;)

@Wassim-AB
Copy link
Author

@oknozor Do you have any idea about the error ?

@oknozor
Copy link
Collaborator
oknozor commented Jun 21, 2023

Sometimes coverage upload fails, this should not make the CI fail. don't worry though this is not blocking.

@oknozor
Copy link
Collaborator
oknozor commented Jun 23, 2023

Hey @Wassim-AB I think we should also be able to override cog.toml settings using the cli flag --skip-ci as mentioned in #226.
If clippy is not happy you can add an exception using the #[allow(..)] macro for this feature.

@Wassim-AB
Copy link
Author

Hey @oknozor, no problem, will be working on it today ;)

@cocogitto-bot
Copy link
cocogitto-bot bot commented Jun 23, 2023

✔️ fea0ece - Conventional commits check succeeded.

@Wassim-AB
Copy link
Author

Hey @oknozor, I'm done, I've also renamed the option "skip_ci" instead of "skip_ci_pattern" within the cog.toml, let me know if you're ok with it :)

@stefan-vatov
Copy link

Hey @oknozor , this would be very useful to get merged. Anything else that needs to be done to move this forward :) ?

@Wassim-AB
Copy link
Author

Hey @oknozor, will you need some help documenting this feature ? It's fairly easy tbh, it's just out of principle ;)

@oknozor
Copy link
Collaborator
oknozor commented Aug 1, 2023

Hey @Wassim-AB I am merging this now but yes, it would be nice if you could update the doc before next release.

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.

3 participants
0