Skip to content

Remove lib linting CI#1902

Merged
Velin92 merged 1 commit intodevelopfrom
mauroromito/remove_lib_linting
Mar 11, 2025
Merged

Remove lib linting CI#1902
Velin92 merged 1 commit intodevelopfrom
mauroromito/remove_lib_linting

Conversation

@Velin92
Copy link
Copy Markdown
Member

@Velin92 Velin92 commented Mar 11, 2025

Since we don't release anymore the library on cocoapods, and there is an issue with Cocoapods that only allows linting to work with up to Xcode 14.2 with the libbase58 library, even if the project can be built with newer versions... then we decided to just get rid of this workflow entirely.

We could run the linting and just the linting with an older version, but this feels like a waste of resources given the fact that as mentioned above, we don't publish on cocapods anymore

since we don't really release the app anymore on cocoapods, and there is an issue with the libbase58 library that only allows linting to work with up to Xcode14.2, even if the project can build with newer versions.
@Velin92 Velin92 changed the title Removed lib linting CI Remove lib linting CI Mar 11, 2025
Copy link
Copy Markdown
Member

@pixlwave pixlwave left a comment

Choose a reason for hiding this comment

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

🔥🔥🔥

Copy link
Copy Markdown
Contributor

@manuroe manuroe left a comment

Choose a reason for hiding this comment

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

It is fine by me.
Thanks for the cleanup.

@Velin92 Velin92 merged commit 6a49ee7 into develop Mar 11, 2025
1 of 2 checks passed
@Velin92 Velin92 deleted the mauroromito/remove_lib_linting branch March 11, 2025 18:25
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