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

Update auto comment workflow due to deprecation warnings #1126

Merged
merged 1 commit into from
Jan 15, 2024

Conversation

ErdemOzgen
Copy link
Contributor

  • This will fix deprecation warnings in auto comment
  • For github_token and Docker username and password I do not have enough permission But we can try these.
  1. Disable and re-enable the integration: Go to the repository's settings, navigate to the "Integrations & services" (or similar) section, find the GitHub Actions integration, and disable it. Then, re-enable it again. This can help refresh the integration's access to the repository.

  2. Check repository permissions: Ensure that the GitHub Actions integration has the necessary permissions to access the repository and perform the required actions. Make sure it has appropriate read and write access.

  3. Review workflow configuration: Double-check your workflow file to ensure that it references the correct repository name and settings. Confirm that the release package and tag you are trying to access are properly set up in the workflow.

  4. Verify authentication: Ensure that the GITHUB_TOKEN used in your workflow file is correctly set and has the necessary permissions to access and modify releases.

image

Adding Docker Credentials: To securely add your Docker username and password to GitHub repository for CI/CD processes:

  1. Go to your repository on GitHub.
  2. Click on 'Settings' and then 'Secrets'.
  3. Choose 'New repository secret'.
  4. Add your Docker username and password as two separate secrets. Name them DOCKER_USERNAME and DOCKER_PASSWORD.
  5. Use these secrets in your GitHub Actions workflow by referencing them as ${{ secrets.DOCKER_USERNAME }} and ${{ secrets.DOCKER_PASSWORD }}.

Copy link
Contributor

@AnonymousWP AnonymousWP left a comment

Choose a reason for hiding this comment

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

Approved. @yogeshojha Assigned you as assignee and reviewer for reviewing this PR and checking the Docker build-related issue.

@AnonymousWP
Copy link
Contributor

I'm waiting with merging before Yogesh replied/reviewed.

@yogeshojha yogeshojha merged commit 36fb41d into yogeshojha:master Jan 15, 2024
3 of 5 checks passed
@AnonymousWP
Copy link
Contributor

@yogeshojha Have you also followed the steps in the initial post? Merging alone is not enough.

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