chore: Replace deprecated command with environment file

[jongwooo]

SUMMARY

Fixes #24063 (comment)
CC @geido

BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

TESTING INSTRUCTIONS

ADDITIONAL INFORMATION

• Has associated issue:
• Required feature flags:
• Changes UI
• Includes DB Migration (follow approval process in SIP-59)
  • Migration is atomic, supports rollback & is backwards-compatible
  • Confirm DB migration upgrade and downgrade tested
  • Runtime estimates and downtime expectations provided
• Introduces new feature or API
• Removes existing feature or API

[dpgaspar]

./scripts/ci_check_no_file_changes.sh is shell script so the unit test is being skipped here.

Run ./scripts/ci_check_no_file_changes.sh python
Searching for changes in python files

CHANGED FILES:
scripts/tag_latest_release.sh

No changes detected... Exiting with SUCCESS code

[dpgaspar]

./scripts/ci_check_no_file_changes.sh is shell script so the unit test is being skipped here.

Run ./scripts/ci_check_no_file_changes.sh python
Searching for changes in python files

CHANGED FILES:
scripts/tag_latest_release.sh

No changes detected... Exiting with SUCCESS code

here this should help: #27303

[jongwooo]

here this should help: #27303

@dpgaspar Great catch!

[dpgaspar]

ok #27303 is merged, can you rebase, so we can catch the failing test please

[jongwooo]

ok #27303 is merged, can you rebase, so we can catch the failing test please

@dpgaspar Sure :)

[dpgaspar]

ok you caught it!

Change: https://github.com/apache/superset/blob/master/tests/unit_tests/scripts/tag_latest_release_test.py#L47 to:
The tag does_not_exist does not exist. Please use a different tag.\n

[jongwooo]

@dpgaspar Should I change line 36 as well?

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 69.57%. Comparing base (7854b62) to head (c7f1738).
Report is 1475 commits behind head on master.

Additional details and impacted files

@@            Coverage Diff             @@
##           master   #27304      +/-   ##
==========================================
+ Coverage   67.33%   69.57%   +2.24%     
==========================================
  Files        1908     1908              
  Lines       74544    74544              
  Branches     8314     8314              
==========================================
+ Hits        50196    51866    +1670     
+ Misses      22296    20626    -1670     
  Partials     2052     2052              

Flag	Coverage Δ
mysql	78.00% <ø> (ø)
postgres	78.10% <ø> (ø)
python	82.88% <ø> (+4.64%)	⬆️
sqlite	77.62% <ø> (ø)
unit	56.50% <ø> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[dpgaspar]

@dpgaspar Should I change line 36 as well?

probably, test it

[dpgaspar]

Nice! thank you for doing this

[eschutho]

@jongwooo Thanks for this fix! I think we may need to update this line here as well from SKIP_TAG::false to SKIP_TAG=false. Do you mind adding one more PR with this other change?

[jongwooo]

@eschutho That'd be great.

superset/scripts/build_docker.py

Line 82 in ad6327d

return "SKIP_TAG::false" in output

right?