Skip to content

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

Closed
darkdragon-001 wants to merge 1 commit intoflatpak:masterfrom
darkdragon-001:artifact-flatpak-suffix
Closed

Include ".flatpak" suffix in artifact name to avoid conflicts with other artifacts#177
darkdragon-001 wants to merge 1 commit intoflatpak:masterfrom
darkdragon-001:artifact-flatpak-suffix

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

@bilelmoussaoui
Copy link
Member

Pulled in #224, thanks & sorry for the delay.

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