Skip to content

update TagBot #36

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

Merged
merged 1 commit into from
Jan 17, 2021
Merged

update TagBot #36

merged 1 commit into from
Jan 17, 2021

Conversation

jishnub
Copy link
Member

@jishnub jishnub commented Nov 24, 2020

Switch to the suggested workflow to work around github disabling actions on inactive repos.

runs-on: ubuntu-latest
steps:
- uses: JuliaRegistries/TagBot@v1
with:
token: ${{ secrets.GITHUB_TOKEN }}
ssh: ${{ secrets.DOCUMENTER_KEY }}
Copy link
Member

@johnnychen94 johnnychen94 Nov 24, 2020

Choose a reason for hiding this comment

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

This is specifically needed when unit test CI is run on github action. This ssh key helps work around the limitation that "one github action cannot trigger another one". MappedArrays is still using travis so we don't need this.

Most importantly, I believe for this repo, DOCUMENTER_KEY key isn't set.

Copy link
Member Author

@jishnub jishnub Nov 24, 2020

Choose a reason for hiding this comment

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

You're right, but do you plan to move away from travis at some point? I had retained this as a future-proof command

Copy link
Member

@johnnychen94 johnnychen94 Nov 24, 2020

Choose a reason for hiding this comment

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

I believe we should but given that this package isn't actively developed, this is of low priority. I don't have write access to this repo so we should leave it to @timholy.

@timholy timholy merged commit ca07b29 into JuliaArrays:master Jan 17, 2021
@timholy
Copy link
Member

timholy commented Jan 17, 2021

I added the key.

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.

3 participants