Skip to content
This repository has been archived by the owner on Oct 13, 2023. It is now read-only.

chore: bump @actions deps #222

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

duskmoon314
Copy link

Upgrade deps:

  • @actions/core: ^1.10.0
  • @actions/exec: ^1.1.1
  • @actions/io: ^1.1.2

close #221

Notice

The lock file version is old, and npm v8 has upgraded its format. Therefore, to avoid conflict, I split this PR into three commits, and the latter two commits can be removed if unnecessary.

Feel free to edit this description.

@weikengchen
Copy link

It would be great if this can be approved soon.

arnaudgolfouse added a commit to arnaudgolfouse/Dijkstra_map_for_Godot that referenced this pull request Oct 23, 2022
This suppresses some deprecation messages. For now, deprecation message are also emmited when using `action-rs` actions. They should resolve this soon with <actions-rs/toolchain#222>.
arnaudgolfouse added a commit to arnaudgolfouse/Dijkstra_map_for_Godot that referenced this pull request Oct 23, 2022
This suppresses some deprecation messages. For now, deprecation message are also emmited when using `action-rs` actions. They should resolve this soon with <actions-rs/toolchain#222>.
arnaudgolfouse added a commit to arnaudgolfouse/Dijkstra_map_for_Godot that referenced this pull request Oct 23, 2022
This suppresses some deprecation messages. For now, deprecation message are also emmited when using `action-rs` actions. They should resolve this soon with <actions-rs/toolchain#222>.
astrale-sharp pushed a commit to MatejSloboda/Dijkstra_map_for_Godot that referenced this pull request Oct 24, 2022
This suppresses some deprecation messages. For now, deprecation message are also emmited when using `action-rs` actions. They should resolve this soon with <actions-rs/toolchain#222>.
@AntoniosBarotsis
Copy link

It seems that even though this project has been left unmaintained for a while, many people still care about it and use it a lot. Maybe we could make a fork and start making security patches there, I'd be interested in helping maintain that.

There are a lot of PRs that have been open for almost 3 years now...

@duskmoon314
Copy link
Author

It seems that even though this project has been left unmaintained for a while, many people still care about it and use it a lot. Maybe we could make a fork and start making security patches there, I'd be interested in helping maintain that.

There are a lot of PRs that have been open for almost 3 years now...

I think this has been discussed in #216 and https://www.reddit.com/r/rust/comments/vyx4oj/actionsrs_organization_became_unmaintained/. Maybe you will have an interest there.

For others who want to try an alternative: Currently, I am testing invoke rustup directly in GitHub actions and it seems to be working fine. Also, there is dtolnay's rust-toolchain action. (Maybe named as dtolnay/rust-toolchain)

@AntoniosBarotsis
Copy link

Thanks a lot!

bors bot added a commit to nervosnetwork/ckb that referenced this pull request Nov 11, 2022
3690: ci: The set-output command is deprecated and will be disabled soon, f… r=zhangsoledad a=liya2017

<!--
Thank you for contributing to nervosnetwork/ckb!

If you haven't already, please read [CONTRIBUTING](https://github.com/nervosnetwork/ckb/blob/develop/CONTRIBUTING.md) document.

If you're unsure about anything, just ask; somebody should be along to answer within a day or two.

### What is changed and how it works?

Proposal: [xxx](url) <!-- REMOVE this line if not applicable -->

What's Changed:

### Related changes
- Fixed the warning that the set-output command  is deprecated.
- Still have some warning about set-output command since the  https://github.com/actions-rs
[toolchain](https://github.com/actions-rs/toolchain)  need to bump `@actions` deps. A PR actions-rs/toolchain#222 to filx it ,but not merge yet.

### Check List <!--REMOVE the items that are not applicable-->

Tests <!-- At least one of them must be included. -->
- No code ci-runs-only: [ quick_checks,linters ]


### Release note <!-- Choose from None, Title Only and Note. Bugfixes or new features need a release note. -->

```release-note
None: Exclude this PR from the release note.
```


Co-authored-by: liya2017 <Liya@cryptape.com>
yvt added a commit to r3-os/actions-rust-toolchain that referenced this pull request Nov 12, 2022
yvt added a commit to r3-os/r3 that referenced this pull request Nov 12, 2022
Includes the following new unmerged PRs:

- <actions-rs/toolchain#220> "Update the action
  to run on Node 16"
- <actions-rs/toolchain#222> "chore: bump
  @actions deps"

Addresses the "Node.js 12 actions are deprecated" and "`set-output`
command is deprecated" warnings.
@Mart-Bogdan
Copy link

PLEASE merge this! Someone.

@svartalf if you could, please.

@Mart-Bogdan
Copy link

Perhaps this organisation access could be given to someone? :-(

@mokeyish
Copy link

mokeyish commented Dec 3, 2022

@svartalf ,is the only one who can run the organization?

deep-soft added a commit to deep-soft/rust-toolchain-actions-rs that referenced this pull request Oct 6, 2023
This PR supersedes actions-rs#220 and actions-rs#222 by applying both fixes at once
Fixes actions-rs#219
Fixes actions-rs#221

You can now use this action (and cargo) without deprecations by using a workflow similar to this:

on: [push]

name: build

jobs:
  check:
    name: Rust project
    runs-on: ubuntu-latest
    steps:
      - uses: actions/checkout@v3
      - name: Install latest nightly
        uses: ThexXTURBOXx/toolchain@master
        with:
            toolchain: nightly
            override: true
            components: rustfmt, clippy

      # `cargo check` command here will use installed `nightly`
      # as it is set as an "override" for current directory

      - name: Run cargo check
        uses: richb-hanover/cargo@master
        with:
          command: check
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Development

Successfully merging this pull request may close these issues.

set-output command is deprecated
5 participants