(superset) 03/04: dry

[ma...@apache.org]

This is an automated email from the ASF dual-hosted git repository.

maximebeauchemin pushed a commit to branch pyproject.toml
in repository https://gitbox.apache.org/repos/asf/superset.git

commit f232bc1a8fd9f9b368ef89ecab6d1c8153271c55
Author: Maxime Beauchemin <ma...@gmail.com>
AuthorDate: Mon Mar 25 16:29:32 2024 -0700

    dry
---
 .github/supersetbot/src/github.js | 8 +++++---
 1 file changed, 5 insertions(+), 3 deletions(-)

diff --git a/.github/supersetbot/src/github.js b/.github/supersetbot/src/github.js
index b2b0ca2849..7b2eeada1a 100644
--- a/.github/supersetbot/src/github.js
+++ b/.github/supersetbot/src/github.js
@@ -287,10 +287,11 @@ class Github {
       const commitMessage = `chore(🤖): bump python "${changed[0]}"`;
       await runShellCommand({ command: `git commit -m "${commitMessage}"`, cwd, verbose });
 
-      // Push changes
-      await runShellCommand({ command: `git push origin ${branchName}`, cwd, verbose });
-
       if (!dryRun) {
+
+        // Push changes
+        await runShellCommand({ command: `git push origin ${branchName}`, cwd, verbose });
+
         // Create a PR
         this.octokit.pulls.create({
             ...this.unPackRepo(),
@@ -303,6 +304,7 @@ class Github {
             console.log(`Pull request created: ${data.html_url}`);
         })
         .catch(console.error);
+
       }
     }
     // Cleaning up