Skip to content
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

FIX: gh_token scope #1679

Merged
merged 1 commit into from
Dec 7, 2022
Merged

FIX: gh_token scope #1679

merged 1 commit into from
Dec 7, 2022

Conversation

jorgepiloto
Copy link
Member

As soon as you declare the permissions section, everything goes to null:

If you specify the access for any of these scopes, all of those that are not specified are set to none.

We need write scope for the GH_TOKEN to write to the gh-pages branch.

@github-actions github-actions bot added CI/CD Related with CICD, Github Actions, etc Maintenance General maintenance of the repo (libraries, cicd, etc) BUG Issue, problem or error in PyMAPDL labels Dec 7, 2022
@github-actions
Copy link
Contributor

github-actions bot commented Dec 7, 2022

Please add one of the following labels to add this contribution to the Release Notes 👇

@jorgepiloto jorgepiloto merged commit deb224d into main Dec 7, 2022
@jorgepiloto jorgepiloto deleted the fix/permissions branch December 7, 2022 13:38
Revathyvenugopal162 pushed a commit that referenced this pull request Dec 7, 2022
jorgepiloto added a commit that referenced this pull request Dec 8, 2022
* DOC: implement multi-version in main (#1674)

* WIP: migrate to pyansys actions

* WIP: Add release actions

* WIP: rename package name in favour of release actions

* WIP: refactor the deployment

* WIP: refactor the actions order

* WIP: Adapt conf for the multidoc

* resolves code style error

* Update .github/workflows/ci.yml

Co-authored-by: Jorge Martínez <28702884+jorgepiloto@users.noreply.github.com>

* WIP: Change build file for pdf build

* DBG: show latex dir

* CI: apply double quotes

* CI: fix cache doc/_build

* WIP: Change _build in validate png module

* Update theme

* change the html docs name

* Update the actions to 2

* MAINT: use ansys-sphinx-theme==0.8.0

* Add build env

* Add build env

* CI: fix double quotes

* Adapt to double quotes

* fix typo

* DBG: avoid cache

* DBG: avoid cache

* Resolves the package check actions

* CI: update nightly docs

Co-authored-by: Jorge Martínez <28702884+jorgepiloto@users.noreply.github.com>
Co-authored-by: Jorge Martinez Garrido <jorge.martinezgarrido@ansys.com>

* FIX: gh_token scope (#1679)

* Modify package name

* Modify pre-commit

* Modify pre-commit

* revert changes with ci/cd yml

* revert changes with ci/cd yml

* Add reelase docs

* Apply suggestions from code review

* Add correct build path

* Add requirement

* Correcting path for tar logs

* Take out the bot

* FIX: do not update pyproject.toml

Co-authored-by: Jorge Martínez <28702884+jorgepiloto@users.noreply.github.com>
Co-authored-by: Jorge Martinez Garrido <jorge.martinezgarrido@ansys.com>
@germa89
Copy link
Collaborator

germa89 commented Dec 12, 2022

This is a very interesting point!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
BUG Issue, problem or error in PyMAPDL CI/CD Related with CICD, Github Actions, etc Maintenance General maintenance of the repo (libraries, cicd, etc)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants