mirror of
https://github.com/netblue30/firejail.git
synced 2026-05-15 14:16:14 -06:00
[PR #6104] [MERGED] ci: re-enable sort.py #5827
Labels
No labels
LTS merge
LTS merge
bug
bug
converted-to-discussion
doc-todo
documentation
duplicate
enhancement
file-transfer
firecfg
firejail-in-firejail
firetools
graphics
help wanted
information_old
installation
invalid
modif
moved
needinfo
networking
notabug
notourbug
old-version
overlayfs
packaging
profile-request
pull-request
question
question_old
removal
runtime-permissions
sandbox-ipc
security
stale
wiki
wiki
wontfix
wordpress
workaround
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference: github-starred/firejail#5827
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
📋 Pull Request Information
Original PR: https://github.com/netblue30/firejail/pull/6104
Author: @kmk3
Created: 11/25/2023
Status: ✅ Merged
Merged: 12/4/2023
Merged by: @netblue30
Base:
master← Head:ci-enable-sort-py📝 Commits (1)
78bbcb7ci: re-enable sort.py📊 Changes
1 file changed (+4 additions, -4 deletions)
View changed files
📝
.github/workflows/check-profiles.yml(+4 -4)📄 Description
It was disabled on commit
df6ea884f("merges, disable sort.py in profilechecks temporarely, two more private-etc profiles", 2023-02-14).
Currently all profiles are sorted and there are no ongoing
private-etcchanges, so it should be safe to re-enable.
Note that the script is useful to catch sorting issues not only in
private-etcbut also in other commands, such asseccomp[1] [2].This is a follow-up to #6070.
Relates to #5610.
[1] https://github.com/netblue30/firejail/pull/6066#discussion_r1372055800
[2] https://github.com/netblue30/firejail/pull/6067#discussion_r1372027243
🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.