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

[BUG] Secure variables only for own repositories/PRs #90

Open
guibranco opened this issue Jul 19, 2023 · 0 comments · Fixed by #93
Open

[BUG] Secure variables only for own repositories/PRs #90

guibranco opened this issue Jul 19, 2023 · 0 comments · Fixed by #93
Assignees
Labels
🐛 bug Issues related to bugs or errors ⚙️ CI/CD Continuous Integration/Continuous Deployment processes enhancement New feature or request

Comments

@guibranco
Copy link
Member

guibranco commented Jul 19, 2023

Description

When the build runs from a forked repository, it fails when it needs to use a secure environment variable.
Conditionally execute these steps only if not in a forked pull request.

Screenshots

- ps: if(-Not $env:FORKED_DETECTION) {
    // Execute the action
    codeclimate-test-reporter upload-coverage -i "Tests\%SOLUTION_NAME%.UnitTests\code-climate.json" -r %CODECLIMATE_TOKEN% 
}

Additional information

Maybe we can use the APPVEYOR_PULL_REQUEST_HEAD_REPO_NAME environment variable to detect the repository where the PR comes from

@guibranco guibranco self-assigned this Jul 19, 2023
@guibranco guibranco added ⚙️ CI/CD Continuous Integration/Continuous Deployment processes 🐛 bug Issues related to bugs or errors enhancement New feature or request labels Jul 19, 2023
@guibranco guibranco linked a pull request Jul 23, 2023 that will close this issue
@guibranco guibranco reopened this Jul 25, 2024
@guibranco guibranco changed the title Secure variables only for own repositories/prs [BUG] Secure variables only for own repositories/PRs Jul 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🐛 bug Issues related to bugs or errors ⚙️ CI/CD Continuous Integration/Continuous Deployment processes enhancement New feature or request
Development

Successfully merging a pull request may close this issue.

1 participant