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

Revert 63 add-to-project action #92

Merged
merged 1 commit into from
Nov 27, 2023
Merged

Revert 63 add-to-project action #92

merged 1 commit into from
Nov 27, 2023

Conversation

MaxJa4
Copy link
Collaborator

@MaxJa4 MaxJa4 commented Nov 23, 2023

Reverts #90 as GITHUB_TOKEN doesn't have the rights to edit projects.

Solving this requires a repo admin to add ADD_TO_PROJECT_PAT as token in this repo's settings, like described here.

This comment was marked as off-topic.

@ll7
Copy link
Member

ll7 commented Nov 27, 2023

The PAT will expire on 2024-01-11.

@MaxJa4 MaxJa4 merged commit f970b1c into main Nov 27, 2023
3 checks passed
@MaxJa4 MaxJa4 deleted the revert-90-MaxJa4-patch-1 branch November 27, 2023 09:42
@ll7
Copy link
Member

ll7 commented Nov 27, 2023

#104 was successfully added to the project, correct? So the test was successful?

@MaxJa4
Copy link
Collaborator Author

MaxJa4 commented Nov 27, 2023

#104 was successfully added to the project, correct? So the test was successful?

Yes, test successful. We can fix the deprecation warning too with a action version update or just leave it as is - it works either way now.

Copy link
Collaborator

@samuelkuehnel samuelkuehnel left a comment

Choose a reason for hiding this comment

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

Test works, approved!

@ll7
Copy link
Member

ll7 commented Nov 27, 2023

#104 was successfully added to the project, correct? So the test was successful?

Yes, test successful. We can fix the deprecation warning too with a action version update or just leave it as is - it works either way now.

Removing a warning would be nice.

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.

4 participants