Skip to content

Give myself permissions on rust-artwork repo.#2267

Merged
jieyouxu merged 1 commit intorust-lang:mainfrom
m-ou-se:art
Feb 25, 2026
Merged

Give myself permissions on rust-artwork repo.#2267
jieyouxu merged 1 commit intorust-lang:mainfrom
m-ou-se:art

Conversation

@m-ou-se
Copy link
Member

@m-ou-se m-ou-se commented Feb 25, 2026

Nobody currently has access to rust-artwork.

The most relevant thing in this repo is the logo svg, the current version of which I authored five years ago. (rust-lang/rust-artwork#9) I'd like to add the logo in a few more formats and tidy up the repository a bit.

@github-actions
Copy link

github-actions bot commented Feb 25, 2026

Dry-run check results

[WARN  sync_team] sync-team is running in dry mode, no changes will be applied.
[INFO  sync_team] synchronizing crates-io
[INFO  sync_team] synchronizing github
[INFO  sync_team] 💻 Repo Diffs:
    📝 Editing repo 'rust-lang/bors':
      Rulesets:
          Ruleset for main
            No changes
    📝 Editing repo 'rust-lang/rust-artwork':
      Permission Changes:
        Giving user 'm-ou-se' write permission

@steffahn
Copy link
Member

steffahn commented Feb 25, 2026

AFAICT, "write" is the more common choice here for configuring access these days, rather than "maintain" (though I forgot the exact differences since since last time I checked).

Also I guess the “access is left empty for now” comment becomes outdated with the PR.

@m-ou-se m-ou-se force-pushed the art branch 2 times, most recently from 0ceacae to d93fd40 Compare February 25, 2026 21:31
@m-ou-se
Copy link
Member Author

m-ou-se commented Feb 25, 2026

AFAICT, "write" is the more common choice here for configuring access these days, rather than "maintain" (though I forgot the exact differences since since last time I checked).

They are almost the same, but "maintain" allows me to adjust the repo description.

Also I guess the “access is left empty for now” comment becomes outdated with the PR.

Updated.

@Kobzol
Copy link
Member

Kobzol commented Feb 25, 2026

The description should be set through team 😅 Or the automation will just overwrite it on the next sync.

description = "Cool artworkz"

in the repo TOML file (it's already set there).

@m-ou-se
Copy link
Member Author

m-ou-se commented Feb 25, 2026

Oh cool, didn't realize how powerful the team repository had become.

Changed it to "write".

Copy link
Member

@jieyouxu jieyouxu left a comment

Choose a reason for hiding this comment

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

@jieyouxu jieyouxu added this pull request to the merge queue Feb 25, 2026
Merged via the queue into rust-lang:main with commit 908a30f Feb 25, 2026
3 checks passed
@m-ou-se m-ou-se deleted the art branch February 26, 2026 11:30
@ehuss
Copy link
Contributor

ehuss commented Feb 26, 2026

@jieyouxu I don't have any objection to giving @m-ou-se access, but our policy is to never give individual access to repositories. This probably should have been given to a team (social-media or council?, idk).

@jieyouxu
Copy link
Member

jieyouxu commented Feb 26, 2026

@jieyouxu I don't have any objection to giving @m-ou-se access, but our policy is to never give individual access to repositories. This probably should have been given to a team (social-media or council?, idk).

Probably. We can change it to just council? Also a question: was that actually written down somewhere? I tried to cite that policy but couldn't find it last time

@m-ou-se
Copy link
Member Author

m-ou-se commented Feb 26, 2026

social-media probably makes more sense than council. The council is supposed to have power as a group, but not necessarily as indivivduals. Giving power to members of the council individually isn't super useful, because membership rotates. It would be annoying if I lose my acceses to rust-artwork if someone else starts representing libs on the council.

Happy to send a PR that changes it to social-media for now. Adding a T-logo team seems a bit overkill. ^^

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.

5 participants