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

Create deb and rpm automatically #25

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

Conversation

samuel-w
Copy link
Contributor

@samuel-w samuel-w commented Jan 14, 2024

Also fixed the test (which can get rate limited somehow even though it's on GitHub's servers, but shouldn't happen to users unless they spam-click the buttons) and bumped action versions
https://github.com/samuel-w/OpenLauncher/releases/tag/v0.3.2

Closes #24

@samuel-w samuel-w marked this pull request as draft January 15, 2024 06:59
@samuel-w
Copy link
Contributor Author

samuel-w commented Jan 15, 2024

Just realized it should be packaged without being self-contained.
Edit: Fixed!

@samuel-w samuel-w marked this pull request as ready for review January 15, 2024 17:54
@IntelOrca
Copy link
Collaborator

When is the job supposed to run? It isn't part of the commit or tag actions.

@samuel-w
Copy link
Contributor Author

It runs when you push a tag that starts with v, just like the current release action is set up.

@IntelOrca
Copy link
Collaborator

It runs when you push a tag that starts with v, just like the current release action is set up.

I am just wondering, why not create the package every build?

@IntelOrca
Copy link
Collaborator

@samuel-w Notifying you again in case you missed the last message

- name: Checkout
uses: actions/checkout@v3
- name: Setup .NET Core SDK
uses: actions/setup-dotnet@v3
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
uses: actions/setup-dotnet@v3
uses: actions/setup-dotnet@v4

src/openlauncher/bin/Release/net7.0/${{ matrix.rid }}/publish/openlauncher.exe
steps:
- name: Checkout
uses: actions/checkout@v3
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
uses: actions/checkout@v3
uses: actions/checkout@v4

@@ -17,9 +19,9 @@ jobs:
- name: Checkout
uses: actions/checkout@v3
- name: Setup .NET Core SDK
uses: actions/setup-dotnet@v1
uses: actions/setup-dotnet@v3
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
uses: actions/setup-dotnet@v3
uses: actions/setup-dotnet@v4

path: src/openlauncher/bin/Release/net7.0/${{ matrix.rid }}/publish/**/*
- name: Create release
uses: softprops/action-gh-release@v1
path: src/openlauncher/bin/Release/net${{ env.dotnet-version }}/${{ matrix.rid }}/publish/**/*
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
path: src/openlauncher/bin/Release/net${{ env.dotnet-version }}/${{ matrix.rid }}/publish/**/*
path: src/openlauncher/bin/Release/net${{ env.dotnet-version }}/${{ matrix.rid }}/publish/**/*
if-no-files-found: error

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.

Create .desktop file and deb and rpm Packages
3 participants