Skip to content

Commit c849394

Browse files
removed unnecessary code and reverted slack temperory code
1 parent 228e592 commit c849394

2 files changed

Lines changed: 2 additions & 19 deletions

File tree

.github/workflows/release-make-it-native.yml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -32,7 +32,7 @@ jobs:
3232
run: node scripts/release-make-it-native.mjs
3333

3434
- name: Send success message
35-
if: false # Disabled for testing, restore to success() when done
35+
if: success()
3636
uses: ./.github/actions/slack-notification
3737
with:
3838
channel-id: ${{ secrets.SLACK_CHANNEL_ID }}
@@ -45,7 +45,7 @@ jobs:
4545
bot-token: ${{ secrets.SLACK_BOT_TOKEN }}
4646

4747
- name: Send failure message
48-
if: false # Disabled for testing, restore to failure() when done
48+
if: failure()
4949
uses: ./.github/actions/slack-notification
5050
with:
5151
channel-id: ${{ secrets.SLACK_CHANNEL_ID }}

scripts/release-make-it-native.mjs

Lines changed: 0 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -121,23 +121,6 @@ function injectUnreleasedToDoc(docPath, unreleasedContent) {
121121
return `${frontmatter}\n\n${firstParagraph}\n${title}\n\n${unreleasedContent}\n\n${afterFirstParagraph}`;
122122
}
123123

124-
// async function syncForkWithUpstream() {
125-
// // Sync the fork's development branch with upstream (mendix/docs) via the GitHub API.
126-
// // This is equivalent to clicking "Sync fork" on GitHub and ensures the fork
127-
// // doesn't have extra commits (e.g., from sync.yml merge commits) that would
128-
// // appear in the PR diff.
129-
// console.log("Syncing MendixMobile/docs fork with upstream mendix/docs...");
130-
// const result = await octokit.request(
131-
// "POST /repos/{owner}/{repo}/merge-upstream",
132-
// {
133-
// owner: DOCS_REPO_OWNER,
134-
// repo: DOCS_REPO_NAME,
135-
// branch: "development",
136-
// }
137-
// );
138-
// console.log(`✅ Fork synced: ${result.data.message} (${result.data.merge_type})`);
139-
// }
140-
141124
// This file exists only in the fork (MendixMobile/docs) and not in upstream (mendix/docs).
142125
// Removing it in our branch ensures it doesn't appear in the cross-fork PR diff.
143126
const FORK_SYNC_FILE = ".github/workflows/sync.yml";

0 commit comments

Comments
 (0)