Skip to content

refactor: reorganise metta publish.py to prepare for downstream refactor #4415

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.

Sign up for GitHub

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

Jump to bottom
Merged
relh merged 1 commit into from
Dec 20, 2025
Merged

refactor: reorganise metta publish.py to prepare for downstream refactor #4415

relh merged 1 commit into from
Dec 20, 2025

Conversation

@yatharth
Copy link
Contributor

@yatharth yatharth commented Dec 18, 2025 •
edited
Loading

What this PR does

  • Reorders the functions in publish.py
  • Removes unused EXPECTED_REMOTE_URL constant
  • NO other logic changes

Why it does that

Makes diffs for downstream PRs cleaner.

I made a large refactor and wanted to split it into small, easy-to-review PRs.

Asana Task

@yatharth yatharth changed the title refactor: reorder functions in metta publish.py to match invocation o… refactor: reorganise metta publish.py to prepare for downstream refactor Dec 18, 2025
@yatharth yatharth force-pushed the yatharth/refactor-metta-publish branch from 4e9f4f4 to 9f77024 Compare December 18, 2025 03:15
@yatharth Graphite App
Copy link
Contributor Author

yatharth commented Dec 18, 2025 •
edited
Loading

This stack of pull requests is managed by Graphite. Learn more about stacking.

@yatharth yatharth force-pushed the yatharth/refactor-metta-publish branch from 9f77024 to 1833a0f Compare December 18, 2025 03:21
This was referenced Dec 18, 2025
Merged
Open
This was referenced Dec 18, 2025
Open
Open
@yatharth yatharth force-pushed the yatharth/refactor-metta-publish branch from 1833a0f to adb8acc Compare December 18, 2025 05:01
This was referenced Dec 18, 2025
Open
Merged
Open
Open
@relh relh added this pull request to the merge queue Dec 20, 2025
Merged via the queue into main with commit 6cb49f1 Dec 20, 2025
17 checks passed
@relh relh deleted the yatharth/refactor-metta-publish branch December 20, 2025 23:22
This was referenced Dec 21, 2025
Closed
Merged
This was referenced Dec 23, 2025
Open
Open
Open
Open
Open
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@nishu-builder nishu-builder nishu-builder approved these changes

Assignees

@yatharth yatharth

@nishu-builder nishu-builder

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@yatharth @nishu-builder @relh