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

feat: skip Ignite app load for some commands (backport #3972) #3975

Merged
merged 3 commits into from
Feb 16, 2024

Conversation

mergify[bot]
Copy link
Contributor

@mergify mergify bot commented Feb 16, 2024

This is an automatic backport of pull request #3972 done by Mergify.


Mergify commands and options

More conditions and actions can be found in the documentation.

You can also trigger Mergify actions by commenting on this pull request:

  • @Mergifyio refresh will re-evaluate the rules
  • @Mergifyio rebase will rebase this PR on its base branch
  • @Mergifyio update will merge the base branch into this PR
  • @Mergifyio backport <destination> will backport this PR on <destination> branch

Additionally, on Mergify dashboard you can:

  • look at your merge queues
  • generate the Mergify configuration with the config editor.

Finally, you can contact us on https://mergify.com

* feat: skil Ignite app load for some commands

* chore: update changelog

* ci: fix linting issue

* fix: change app load check to validate level one commands only

All of the runnable commands that should not load Ignite apps are level
one commands (`ignite` command children), so a simple check is enough.

* chore: fix comment

---------

Co-authored-by: Danilo Pantani <danpantani@gmail.com>
(cherry picked from commit 82ec261)
Copy link

codecov bot commented Feb 16, 2024

Codecov Report

Attention: 3 lines in your changes are missing coverage. Please review.

Comparison is base (c33089e) 24.80% compared to head (1b2b561) 24.80%.

❗ Current head 1b2b561 differs from pull request most recent head 90377f3. Consider uploading reports for the commit 90377f3 to get more accurate results

Additional details and impacted files

Impacted file tree graph

@@                 Coverage Diff                 @@
##           release/v28.x.y    #3975      +/-   ##
===================================================
- Coverage            24.80%   24.80%   -0.01%     
===================================================
  Files                  296      296              
  Lines                24904    24906       +2     
===================================================
  Hits                  6178     6178              
- Misses               18160    18162       +2     
  Partials               566      566              
Files Coverage Δ
ignite/cmd/cmd.go 5.75% <0.00%> (-0.07%) ⬇️

changelog.md Outdated Show resolved Hide resolved
changelog.md Outdated Show resolved Hide resolved
@julienrbrt julienrbrt merged commit cc88e20 into release/v28.x.y Feb 16, 2024
8 of 41 checks passed
@julienrbrt julienrbrt deleted the mergify/bp/release/v28.x.y/pr-3972 branch February 16, 2024 23:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component:ci CI/CD workflow and automated jobs. component:cmd component:configs
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants