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

Gh release fix #787

Merged
merged 5 commits into from
Sep 22, 2023
Merged

Gh release fix #787

merged 5 commits into from
Sep 22, 2023

Conversation

CyonAlexRDX
Copy link
Contributor

@CyonAlexRDX CyonAlexRDX commented Sep 22, 2023

  • Reverted accidental error where I used a function instead of variable.... rev_changelog instead of changelog
  • Writing a function which tries everything we can try to produce/locate DSYMs

Successfully produced GH release with DSYMs

Screenshot 2023-09-22 at 15 15 46


sh('git fetch --tags')
most_recent_tags = sh("git tag | grep #{flavour} | sort -r | head -2").split("\n")

changelog = rev_changelog(flavour, most_recent_tags)
versions_string = dependencies_with_hyperlinks()

release_description = "**Dependencies**: \n" + versions_string + "\n\n**Changelog**: \n" + rev_changelog
Copy link
Contributor Author

Choose a reason for hiding this comment

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

oops, I accidentally tried to concatenate a function! not a variable... 🤦‍♂️ untyped langauges 🙃🙃🙃

false
end

private_lane :locate_dsym do |options|
Copy link
Contributor Author

Choose a reason for hiding this comment

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

try try try :D

@@ -42,7 +42,7 @@ platform :ios do
build_number: build_number
)

build_app(
build_ios_app(
Copy link
Contributor Author

Choose a reason for hiding this comment

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

bit cleaner imo

@CyonAlexRDX CyonAlexRDX merged commit 40ee3a6 into main Sep 22, 2023
4 checks passed
@CyonAlexRDX CyonAlexRDX deleted the gh_release_fix branch September 22, 2023 13:51
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants