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

[Bot] Update docs for latest fastlane release 2.213.0 (actions.md, available-plugins.md) 🚀 #1197

Merged
merged 2 commits into from
May 25, 2023

Conversation

joshdholtz
Copy link
Member

Auto-generated by fastlane bot 🤖

Copy link
Member

@rogerluan rogerluan left a comment

Choose a reason for hiding this comment

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

This PR LGTM 💪 I'll fix the doc regression shortly

Comment on lines 528 to 531
openssl aes-256-cbc -k "<password>" -in "<fileYouWantToDecryptPath>" -out "<decryptedFilePath>" -a -d -md [md5|sha256]
openssl aes-256-cbc -k "<password>" -in "<fileYouWantToDecryptPath>" -out "<decryptedFilePath>" -a -d
```

_**Note:** You may need to swap double quotes `"` for single quotes `'` if your match password contains an exclamation mark `!`._
Copy link
Member

Choose a reason for hiding this comment

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

This is my bad, I approved & merged this here: #1196 without noticing that the changes were made on /generated/ directory 😬

I'll get this fixed in fastlane/fastlane.

Copy link
Member

Choose a reason for hiding this comment

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

Fixed here: fastlane/fastlane#21302

Instead of moving forward with this PR as is (causing a regression in the docs), and waiting for the next fastlane deploy to re-generate these docs changes as expected, I'm gonna apply a one-off fix to revert the changes in here, so we can merge these changes, and then next time fastlane gets deployed (assuming it includes fastlane/fastlane#21302 😄 ), everything will be back on track 😃

Copy link
Member

Choose a reason for hiding this comment

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

There: 148ddcb

@rogerluan
Copy link
Member

Netlify is knowingly broken, but this PR looks good to go. I'll merge it in :)

@rogerluan rogerluan merged commit 5fd3472 into master May 25, 2023
3 of 4 checks passed
@rogerluan rogerluan deleted the update-actions-md-1684937766 branch May 25, 2023 03:46
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.

None yet

2 participants