Skip to content

chore: slack_train is a plugin with actions, not official actions#1302

Merged
iBotPeaches merged 1 commit intomasterfrom
remove-slack-train-actions
Apr 17, 2026
Merged

chore: slack_train is a plugin with actions, not official actions#1302
iBotPeaches merged 1 commit intomasterfrom
remove-slack-train-actions

Conversation

@iBotPeaches
Copy link
Copy Markdown
Member

@iBotPeaches iBotPeaches commented Jan 8, 2026

The removal logic for auto-generated actions is not the best. It will keep them up to date, but in the sense of removal - the file will never be updated on fastlane/docs side.

So when the PR comes across during doc updates (https://github.com/fastlane/docs/pull/1299/changes) it builds on top of the previous docs but not with removals.

So I'm under impression that I can remove the outdated slack_train action stuff here, because we already have the slack_train plugin being parsed.

@iBotPeaches iBotPeaches marked this pull request as ready for review January 8, 2026 12:11
@iBotPeaches iBotPeaches requested a review from rogerluan January 20, 2026 14:26
@iBotPeaches iBotPeaches force-pushed the remove-slack-train-actions branch from b4cdcb3 to 08043d8 Compare April 17, 2026 11:06
Copy link
Copy Markdown
Collaborator

@lacostej lacostej left a comment

Choose a reason for hiding this comment

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

Is there an issue somewhere about the fact that the removal isn't working automatically?

@iBotPeaches
Copy link
Copy Markdown
Member Author

Is there an issue somewhere about the fact that the removal isn't working automatically?

made one. I did not see one - #1317

@iBotPeaches iBotPeaches merged commit 607eba4 into master Apr 17, 2026
6 checks passed
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