Skip to content

Clarify SSH auth is only for crate registry index, overall cleanup - #73

Merged
chgrv merged 20 commits into
mainfrom
egraven/remove-gitlab-and-ssh-auth
Sep 12, 2025
Merged

Clarify SSH auth is only for crate registry index, overall cleanup#73
chgrv merged 20 commits into
mainfrom
egraven/remove-gitlab-and-ssh-auth

Conversation

@chgrv

@chgrv chgrv commented Aug 15, 2025

Copy link
Copy Markdown
Member

tlater-famedly
tlater-famedly previously approved these changes Aug 15, 2025

@tlater-famedly tlater-famedly left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I like it, if we're doing breaking changes anyway might as well update the secret name.

Comment thread .github/actions/rust-prepare/action.yml Outdated
Comment thread .github/workflows/publish-crate.yml Outdated
Comment thread .github/workflows/publish-crate.yml Outdated
Comment thread .github/actions/rust-prepare/prepare_git.sh
@chgrv
chgrv force-pushed the egraven/remove-gitlab-and-ssh-auth branch 3 times, most recently from 49014c9 to d0a0a6e Compare September 10, 2025 12:20
@chgrv
chgrv marked this pull request as ready for review September 10, 2025 12:21
@chgrv
chgrv requested a review from a team as a code owner September 10, 2025 12:21
@chgrv
chgrv force-pushed the egraven/remove-gitlab-and-ssh-auth branch from d0a0a6e to e20baed Compare September 10, 2025 12:28
@chgrv chgrv changed the title Remove gitlab references, clarify SSH auth is only for crate registry index Clarify SSH auth is only for crate registry index, overall cleanup Sep 10, 2025
@chgrv
chgrv force-pushed the egraven/remove-gitlab-and-ssh-auth branch from f9c8726 to 591649e Compare September 10, 2025 13:39
@chgrv
chgrv force-pushed the egraven/remove-gitlab-and-ssh-auth branch from 591649e to e62d179 Compare September 10, 2025 13:43
Comment thread README.md Outdated
skomski
skomski previously approved these changes Sep 11, 2025
@chgrv
chgrv force-pushed the egraven/remove-gitlab-and-ssh-auth branch from 0b1997f to d2a2c21 Compare September 11, 2025 13:17
jannden
jannden previously approved these changes Sep 11, 2025

@jannden jannden left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Nice, finally getting rid of Gitlab legacy 🥳

Comment thread .github/actions/rust-prepare/prepare.sh Outdated
Comment thread .github/actions/rust-prepare/prepare.sh
Comment thread .github/actions/rust-prepare/prepare.sh
Comment thread .github/workflows/docker-backend.yml
Comment thread .github/workflows/docker-backend.yml
Comment thread .github/workflows/publish-crate.yml Outdated
Comment thread .github/workflows/rust-workflow.yml Outdated
Comment thread .github/workflows/rust-workflow.yml
Comment thread README.md Outdated
@chgrv
chgrv requested a review from sirewix September 12, 2025 06:57
Comment thread README.md Outdated
sirewix
sirewix previously approved these changes Sep 12, 2025
@chgrv
chgrv force-pushed the egraven/remove-gitlab-and-ssh-auth branch from 6e11a33 to 5e41690 Compare September 12, 2025 13:13
@chgrv
chgrv merged commit 5e41690 into main Sep 12, 2025
3 checks passed
@chgrv
chgrv deleted the egraven/remove-gitlab-and-ssh-auth branch September 12, 2025 13:16
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.

Remove remaining gitlab references

5 participants