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

fix: improve ios bundleID regex to stop bundleIDs from throwing incorrectly #357

Merged
merged 2 commits into from
Jan 14, 2025

Conversation

EArminjon
Copy link
Contributor

Description

PR try to fix #352

Type of Change

  • feat -- New feature (non-breaking change which adds functionality)
  • 🛠️ fix -- Bug fix (non-breaking change which fixes an issue)
  • ! -- Breaking change (fix or feature that would cause existing functionality to change)
  • 🧹 refactor -- Code refactor
  • ci -- Build configuration change
  • 📝 docs -- Documentation
  • 🗑️ chore -- Chore

@CLAassistant
Copy link

CLAassistant commented Jan 13, 2025

CLA assistant check
All committers have signed the CLA.

@github-actions github-actions bot added the Needs Attention OP created or responded to issue and it needs attention. label Jan 13, 2025
@EArminjon EArminjon marked this pull request as ready for review January 13, 2025 16:01
@EArminjon EArminjon force-pushed the fix-ios-bundle-id-regex branch from fcf2840 to ccafdbf Compare January 13, 2025 16:03
@EArminjon EArminjon changed the title fix-ios-bundle-id-regex fix: Wrong ios bundleID regex Jan 13, 2025
@russellwheatley russellwheatley changed the title fix: Wrong ios bundleID regex fix: improve ios bundleID regex to stop bundleIDs from throwing incorrectly Jan 13, 2025
@EArminjon
Copy link
Contributor Author

@russellwheatley CI failed weirdly : jobs has been canceled, what can i do to fix that ?

@russellwheatley
Copy link
Member

@EArminjon - I will update the PR 👍

Copy link
Member

@russellwheatley russellwheatley left a comment

Choose a reason for hiding this comment

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

@EArminjon - Pull requests that fork the repo will cause the CI to fail. I've just ran it locally, and it passed. I've updated the contributing documentation to ask users to clone rather than fork 👍

@russellwheatley russellwheatley merged commit b048aad into invertase:main Jan 14, 2025
5 of 8 checks passed
@EArminjon
Copy link
Contributor Author

Thanks a lot, will use that as soon as you release it ;) !

@EArminjon
Copy link
Contributor Author

Hello @russellwheatley ,

Can you release it on pub.dev please :) ?

@EArminjon EArminjon deleted the fix-ios-bundle-id-regex branch January 17, 2025 12:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Needs Attention OP created or responded to issue and it needs attention.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[bug]: Regression - ValidationException: ios - Invalid Apple App Bundle ID
3 participants