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

Upload to build server in publish workflow for nym-vpn-core #1789

Open
wants to merge 19 commits into
base: develop
Choose a base branch
from

Conversation

octol
Copy link
Contributor

@octol octol commented Dec 13, 2024

  • During the nightly build. Append a build number based on the timestamp if it's a -dev build
  • Upload build artifacts to the build server

This change is Reviewable

@octol octol changed the title Add step in publish-nym-vpn-core to upload to build server after publishing Upload to build server in publish workflow for nym-vpn-core Dec 13, 2024
Copy link
Contributor

@pronebird pronebird left a comment

Choose a reason for hiding this comment

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

:lgtm:

Reviewed 1 of 1 files at r1, all commit messages.
Reviewable status: :shipit: complete! all files reviewed, all discussions resolved (waiting on @octol)

@octol octol force-pushed the jon/ci-upload-to-build-server branch 2 times, most recently from 81d4f06 to 5fa37ba Compare December 17, 2024 13:47
Copy link
Contributor

@pronebird pronebird left a comment

Choose a reason for hiding this comment

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

Reviewed 3 of 3 files at r2, all commit messages.
Reviewable status: all files reviewed, 2 unresolved discussions (waiting on @octol)


nym-vpn-core/crates/nym-dbus/Cargo.toml line 7 at r2 (raw file):

license.workspace = true
edition.workspace = true
version = "1.2.0-dev"

Should this not be version.workspace = true?


nym-vpn-core/crates/nym-windows/Cargo.toml line 7 at r2 (raw file):

license.workspace = true
edition.workspace = true
version = "1.2.0-dev"

Same here?

@octol octol force-pushed the jon/ci-upload-to-build-server branch from 5fa37ba to 46eedec Compare December 20, 2024 22:17
@pronebird pronebird marked this pull request as ready for review December 20, 2024 23:08
@octol octol marked this pull request as draft December 23, 2024 16:02
@octol octol force-pushed the jon/ci-upload-to-build-server branch 2 times, most recently from b9bca47 to 15ee3ec Compare January 22, 2025 09:02
@octol octol force-pushed the jon/ci-upload-to-build-server branch from 29431a9 to d816503 Compare January 23, 2025 08:02
@octol octol marked this pull request as ready for review January 23, 2025 11:59
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.

2 participants