Skip to content

fix: add check to see if it is appfnd org#45

Open
betinacosta wants to merge 1 commit intomainfrom
fix/fork-override
Open

fix: add check to see if it is appfnd org#45
betinacosta wants to merge 1 commit intomainfrom
fix/fork-override

Conversation

@betinacosta
Copy link
Copy Markdown
Member

Description

I added a verification to see if the repository url contains application-foundation string, meaning it is on our organization, therefore, isn't a fork.

Related Issue

AFSDK-3123

(Link to the GitHub issue this PR addresses)

Type of Change

Please check the relevant option:

  • Bug fix (non-breaking change that fixes an issue)
  • New feature (non-breaking change that adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Documentation update
  • Code refactoring
  • Dependency update

How to Test

Describe how reviewers can test your changes:

  1. Step 1
  2. Step 2
  3. Expected result

Checklist

Before submitting your PR, please review and check the following:

  • I have read the Contributing Guidelines
  • I have verified that my changes solve the issue
  • I have added/updated automated tests to cover my changes
  • All tests pass locally
  • I have verified that my code follows the Code Guidelines
  • I have updated documentation (if applicable)
  • I have added type hints for all public APIs
  • My code does not contain sensitive information (credentials, tokens, etc.)
  • I have followed Conventional Commits for commit messages

Breaking Changes

If this PR introduces breaking changes, please describe:

  • What breaks
  • Migration path for users
  • Alternative approaches considered

Additional Notes

Add any additional context, screenshots, or information that would help reviewers.

@betinacosta betinacosta requested a review from a team as a code owner April 9, 2026 20:07
@betinacosta betinacosta marked this pull request as draft April 9, 2026 20:07
sync:
runs-on: ["self-hosted"]
if: ${{ !contains(github.server_url, 'github.com') }}
if: ${{ !contains(github.server_url, 'github.com') && contains(github.repository, 'application-foundation') }}
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

The idea is to check if it is from our org, this was the simplest way I could think, let me know if you guys think is too fragile.

@betinacosta betinacosta marked this pull request as ready for review April 9, 2026 20:10
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.

2 participants