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

Add notify action with nixpkgs update #903

Merged
merged 14 commits into from
Nov 1, 2024
Merged

Add notify action with nixpkgs update #903

merged 14 commits into from
Nov 1, 2024

Conversation

kachick
Copy link
Owner

@kachick kachick commented Oct 31, 2024

Resolves GH-846

Preparation for GH-902

@kachick
Copy link
Owner Author

kachick commented Oct 31, 2024

  1. Clone with depth 1
  2. shallow since target commit. Do not depend on fixed datetime

@kachick
Copy link
Owner Author

kachick commented Nov 1, 2024

GraphQL: Resource not accessible by integration (addComment)

cli/cli#8374

Didn't work in PR by default?

@kachick kachick changed the title Add nixpkgs notify news job Add notify action with nixpkgs update Nov 1, 2024
@kachick kachick marked this pull request as ready for review November 1, 2024 14:48
@kachick kachick merged commit 85ee29b into main Nov 1, 2024
12 of 13 checks passed
@kachick kachick deleted the notify-nixpkgs-merge branch November 1, 2024 14:48
@kachick
Copy link
Owner Author

kachick commented Nov 1, 2024

Raising same error as #903 (comment) after merge 🤔

image

@kachick
Copy link
Owner Author

kachick commented Nov 1, 2024

GITHUB_TOKEN Permissions
  Actions: write
  Attestations: write
  Checks: write
  Contents: write
  Deployments: write
  Discussions: write
  Issues: write
  Metadata: read
  Packages: write
  Pages: write
  PullRequests: write
  RepositoryProjects: write
  SecurityEvents: write
  Statuses: write

kachick added a commit that referenced this pull request Nov 1, 2024
@kachick
Copy link
Owner Author

kachick commented Nov 1, 2024

Followed by these commits 85ee29b...c2cddc8

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.

Notify me when nixpkgs merged several commits in several branches
1 participant