Skip to content

Missing permissions validate-documentation.yml#10495

Merged
SoftFever merged 1 commit into
OrcaSlicer:mainfrom
ianalexis:Doc-Action-fix
Aug 23, 2025
Merged

Missing permissions validate-documentation.yml#10495
SoftFever merged 1 commit into
OrcaSlicer:mainfrom
ianalexis:Doc-Action-fix

Conversation

@ianalexis

@ianalexis ianalexis commented Aug 22, 2025

Copy link
Copy Markdown
Collaborator

The new Documentation Validation didn't have permission to write the comment with the errors.

@ianalexis ianalexis marked this pull request as ready for review August 22, 2025 20:16

@SoftFever SoftFever left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM

@SoftFever SoftFever merged commit 15a835a into OrcaSlicer:main Aug 23, 2025
1 check passed
@ianalexis ianalexis deleted the Doc-Action-fix branch September 6, 2025 01:54
@coryrc

coryrc commented Sep 24, 2025

Copy link
Copy Markdown
Contributor

@ianalexis

Copy link
Copy Markdown
Collaborator Author

@ianalexis still doesn't have permission... https://github.com/SoftFever/OrcaSlicer/actions/runs/17965807869/job/51098113564?pr=10835

Yeah, i know. See this pr #10617
I tried to fixed it here but SoftFever dont want it to be commenting everytime so i change it to only make a "Summary".

@coryrc

coryrc commented Sep 24, 2025

Copy link
Copy Markdown
Contributor

Ah I see, sorry I missed your pending change. I looked for a linked Issue first before replying nudge 😉 😜 😄

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