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

github-actions: bump actions/upload-artifact from 3.0.0 to 3.1.0 #7417

Conversation

dependabot[bot]
Copy link
Contributor

@dependabot dependabot bot commented on behalf of github May 20, 2022

Bumps actions/upload-artifact from 3.0.0 to 3.1.0.

Release notes

Sourced from actions/upload-artifact's releases.

v3.1.0

What's Changed

Commits
  • 3cea537 Merge pull request #327 from actions/robherley/artifact-1.1.0
  • 849aa77 nvm use 12 & npm run release
  • 4d39869 recompile with correct ncc version
  • 2e0d362 bump @​actions/artifact to 1.1.0
  • 09a5d6a Merge pull request #320 from actions/dependabot/npm_and_yarn/ansi-regex-4.1.1
  • 189315d Bump ansi-regex from 4.1.0 to 4.1.1
  • d159c2d Merge pull request #297 from actions/dependabot/npm_and_yarn/ajv-6.12.6
  • c26a7ba Bump ajv from 6.11.0 to 6.12.6
  • 6ed6c72 Merge pull request #303 from actions/dependabot/npm_and_yarn/yargs-parser-13.1.2
  • 2aeee26 Bump yargs-parser from 13.1.1 to 13.1.2
  • Additional commits viewable in compare view

Dependabot compatibility score

Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting @dependabot rebase.


Dependabot commands and options

You can trigger Dependabot actions by commenting on this PR:

  • @dependabot rebase will rebase this PR
  • @dependabot recreate will recreate this PR, overwriting any edits that have been made to it
  • @dependabot merge will merge this PR after your CI passes on it
  • @dependabot squash and merge will squash and merge this PR after your CI passes on it
  • @dependabot cancel merge will cancel a previously requested merge and block automerging
  • @dependabot reopen will reopen this PR if it is closed
  • @dependabot close will close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually
  • @dependabot ignore this major version will close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this minor version will close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this dependency will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)

Bumps [actions/upload-artifact](https://github.com/actions/upload-artifact) from 3.0.0 to 3.1.0.
- [Release notes](https://github.com/actions/upload-artifact/releases)
- [Commits](actions/upload-artifact@6673cd0...3cea537)

---
updated-dependencies:
- dependency-name: actions/upload-artifact
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <[email protected]>
@dependabot dependabot bot requested a review from victorjulien as a code owner May 20, 2022 19:32
@dependabot dependabot bot added dependencies Pull requests that update a dependency file github_actions Pull requests that update GitHub Actions code labels May 20, 2022
@codecov
Copy link

codecov bot commented May 20, 2022

Codecov Report

Merging #7417 (fef573a) into master (b6407c4) will decrease coverage by 0.06%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #7417      +/-   ##
==========================================
- Coverage   75.94%   75.87%   -0.07%     
==========================================
  Files         656      656              
  Lines      189916   189917       +1     
==========================================
- Hits       144233   144104     -129     
- Misses      45683    45813     +130     
Flag Coverage Δ
fuzzcorpus 60.39% <ø> (-0.26%) ⬇️
suricata-verify 51.90% <ø> (+<0.01%) ⬆️
unittests 61.08% <ø> (+<0.01%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

@victorjulien victorjulien mentioned this pull request May 23, 2022
@dependabot @github
Copy link
Contributor Author

dependabot bot commented on behalf of github May 24, 2022

Looks like actions/upload-artifact is up-to-date now, so this is no longer needed.

@dependabot dependabot bot closed this May 24, 2022
@dependabot dependabot bot deleted the dependabot/github_actions/actions/upload-artifact-3.1.0 branch May 24, 2022 05:55
jasonish added a commit to jasonish/suricata that referenced this pull request Feb 24, 2025
jasonish added a commit to jasonish/suricata that referenced this pull request Feb 24, 2025
Fixes rustc compiler warning:

     = note: for more information, see <https://doc.rust-lang.org/nightly/edition-guide/rust-2024/static-mut-references.html>
     = note: shared references to mutable statics are dangerous; it's undefined behavior if the static is mutated or if a mutable reference is created for it while the shared reference lives

Ticket: OISF#7417
jasonish added a commit to jasonish/suricata that referenced this pull request Feb 24, 2025
Fixes Rust compiler warning:

     = note: for more information, see <https://doc.rust-lang.org/nightly/edition-guide/rust-2024/static-mut-references.html>
     = note: shared references to mutable statics are dangerous; it's undefined behavior if the static is mutated or if a mutable reference is created for it while the shared reference lives

Ticket: OISF#7417
jasonish added a commit to jasonish/suricata that referenced this pull request Feb 25, 2025
Other than some debug output on initialization we don't take mutable
references to these static mut's.

    = note: for more information, see <https://doc.rust-lang.org/nightly/edition-guide/rust-2024/static-mut-references.html>
    = note: mutable references to mutable statics are dangerous; it's undefined behavior if any other pointer to the static is used or if any other reference is created for the static while the mutable reference lives

Ticket: OISF#7417
jasonish added a commit to jasonish/suricata that referenced this pull request Feb 25, 2025
It doesn't appear to be needed. The vec being cleared is only set once
per run, so never needs to be cleared.

Removes one point where we have to supress the static_mut_refs compiler
warning.

Ticket: OISF#7417
jasonish added a commit to jasonish/suricata that referenced this pull request Feb 25, 2025
Allows us to get rid of the global supression.

Ticket: OISF#7417
jasonish added a commit to jasonish/suricata that referenced this pull request Feb 25, 2025
As references to static mutables are highly discouraged, remove the
global suppressing of the compiler warning. Each use case can be
suppressed as needed.

Ticket: OISF#7417
jasonish added a commit to jasonish/suricata that referenced this pull request Feb 26, 2025
It doesn't appear to be needed. The vec being cleared is only set once
per run, so never needs to be cleared.

Removes one point where we have to supress the static_mut_refs compiler
warning.

Ticket: OISF#7417
jasonish added a commit to jasonish/suricata that referenced this pull request Feb 26, 2025
Allows us to get rid of the global supression.

Ticket: OISF#7417
jasonish added a commit to jasonish/suricata that referenced this pull request Feb 26, 2025
Simply wrapping in a function removes static_mut_refs compiler
warning.

Ticket: OISF#7417
jasonish added a commit to jasonish/suricata that referenced this pull request Feb 26, 2025
As references to static mutables are highly discouraged, remove the
global suppressing of the compiler warning. Each use case can be
suppressed as needed.

Ticket: OISF#7417
victorjulien pushed a commit to victorjulien/suricata that referenced this pull request Feb 27, 2025
It doesn't appear to be needed. The vec being cleared is only set once
per run, so never needs to be cleared.

Removes one point where we have to supress the static_mut_refs compiler
warning.

Ticket: OISF#7417
victorjulien pushed a commit to victorjulien/suricata that referenced this pull request Feb 27, 2025
Allows us to get rid of the global supression.

Ticket: OISF#7417
victorjulien pushed a commit to victorjulien/suricata that referenced this pull request Feb 27, 2025
Simply wrapping in a function removes static_mut_refs compiler
warning.

Ticket: OISF#7417
victorjulien pushed a commit to victorjulien/suricata that referenced this pull request Feb 27, 2025
As references to static mutables are highly discouraged, remove the
global suppressing of the compiler warning. Each use case can be
suppressed as needed.

Ticket: OISF#7417
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file github_actions Pull requests that update GitHub Actions code
Development

Successfully merging this pull request may close these issues.

0 participants