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

Include ".flatpak" suffix in artifact name to avoid conflicts with other artifacts #177

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

darkdragon-001
Copy link

This is necessary as we remove the .flatpak suffix when adding the arch suffix. In workflows where one also builds other artifacts this name might conflict otherwise and it is obvious for users that app-ARCH is supposed to contain Flatpak bundles.

@darkdragon-001 darkdragon-001 force-pushed the artifact-flatpak-suffix branch 2 times, most recently from 4a71646 to 4ee793f Compare January 22, 2024 08:18
@darkdragon-001 darkdragon-001 changed the title Include "-flatpak" suffix in artifact name to avoid conflicts with other artifacts Include ".flatpak" suffix in artifact name to avoid conflicts with other artifacts Jan 22, 2024
@darkdragon-001
Copy link
Author

@bilelmoussaoui friendly ping

@bilelmoussaoui
Copy link
Member

The changes in the commit are not enough. Please see the contributing guidelines

This is necessary as we remove the .flatpak suffix when adding the arch suffix
@darkdragon-001 darkdragon-001 force-pushed the artifact-flatpak-suffix branch from 4ee793f to 52155f5 Compare February 12, 2024 19:17
@darkdragon-001
Copy link
Author

@bilelmoussaoui Thanks, I updated the dist file. Could you please have another look?

@darkdragon-001
Copy link
Author

@bilelmoussaoui Is anything else missing from my side?

@darkdragon-001
Copy link
Author

@barthalion applied this at flathub-infra@5ca6311

@darkdragon-001
Copy link
Author

Added to flathub-infra@5ca6311

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