docs: fix stale core/ directory paths in contribution docs#6111
Merged
Conversation
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Contributor
Automated Reviewer SuggestionsBased on the
|
Contributor
Author
|
cc @Ma77Ball for review |
Ma77Ball
approved these changes
Jul 4, 2026
Ma77Ball
left a comment
Contributor
There was a problem hiding this comment.
Overall LGTM! The doc changes look reasonable and correct.
Yicong-Huang
added a commit
that referenced
this pull request
Jul 4, 2026
### What changes were proposed in this PR? Cleans up leftovers from before the ASF migration, plus two small adjacent doc fixes: - `github.com/Texera/texera` → `github.com/apache/texera` in `CONTRIBUTING.md` (fork + Actions links), `docs/contribution-guidelines/_index.md`, `docs/examples/_index.md`, and the clone URL in `guide-for-developers.md`. - `CONTRIBUTING.md`: PRs are squash-merged into `main`, not `master` (two mentions). - `docs/examples/_index.md`: dropped the dead link to an `examples/` directory that has never existed in this repo. - `SECURITY.md`: "Kubernates PODs" → "Kubernetes pods". ### Any related issues, documentation, discussions? Closes #6107 ### How was this PR tested? Docs-only change. Verified `main` is the default branch and `apache/texera` the canonical repo (`git remote -v`, `.github/workflows/required-checks.yml`), and that `grep -rn "github.com/Texera/texera"` finds no remaining hits outside files handled by #6111. ### Was this PR authored or co-authored using generative AI tooling? Generated-by: Claude Code (Claude Fable 5) --------- (backported from commit f787472) Co-authored-by: Claude Fable 5 <noreply@anthropic.com>
Yicong-Huang
added a commit
that referenced
this pull request
Jul 4, 2026
### What changes were proposed in this PR? Docs still referenced the pre-ASF `core/` source tree. This PR updates every such path to the current layout: | Stale | Current | | --- | --- | | `core/workflow-operator/.../edu/uci/ics/amber/operator` | `common/workflow-operator/.../org/apache/texera/amber/operator` | | `core/gui`, `/core/new-gui` (operator images) | `frontend/src/assets/operator_images` | | `cd core` / `cd core/gui` (test instructions) | repo root / `cd frontend` | | `config/src/main/resources/...` | `common/config/src/main/resources/...` | | `core/amber/.../generator_operator.py` (renamed) | `amber/.../generator_operator_integer.py` | Also removes `docs/contribution-guidelines/micro-services-local-dev.md`: it documents the now-finished `core/micro-services` migration and build scripts that no longer exist; local development is covered by the developer guide and `bin/local-dev.sh`. No other doc links to that page. ### Any related issues, documentation, discussions? Closes #6105 ### How was this PR tested? Docs-only change. Every new path/link target was verified to exist in the repo: ``` ls common/workflow-operator/src/main/scala/org/apache/texera/amber/operator/{regex,huggingFace,map,filter,flatmap,aggregate} ls frontend/src/assets/operator_images ls amber/src/main/python/pytexera/udf/examples/generator_operator_integer.py grep -rn "core/" docs/contribution-guidelines docs/tutorials # no stale hits remain ``` ### Was this PR authored or co-authored using generative AI tooling? Generated-by: Claude Code (Claude Fable 5) (backported from commit 6829fae) Co-authored-by: Claude Fable 5 <noreply@anthropic.com>
Contributor
Author
|
Backport to |
Yicong-Huang
added a commit
to wwong0/texera
that referenced
this pull request
Jul 5, 2026
### What changes were proposed in this PR? Cleans up leftovers from before the ASF migration, plus two small adjacent doc fixes: - `github.com/Texera/texera` → `github.com/apache/texera` in `CONTRIBUTING.md` (fork + Actions links), `docs/contribution-guidelines/_index.md`, `docs/examples/_index.md`, and the clone URL in `guide-for-developers.md`. - `CONTRIBUTING.md`: PRs are squash-merged into `main`, not `master` (two mentions). - `docs/examples/_index.md`: dropped the dead link to an `examples/` directory that has never existed in this repo. - `SECURITY.md`: "Kubernates PODs" → "Kubernetes pods". ### Any related issues, documentation, discussions? Closes apache#6107 ### How was this PR tested? Docs-only change. Verified `main` is the default branch and `apache/texera` the canonical repo (`git remote -v`, `.github/workflows/required-checks.yml`), and that `grep -rn "github.com/Texera/texera"` finds no remaining hits outside files handled by apache#6111. ### Was this PR authored or co-authored using generative AI tooling? Generated-by: Claude Code (Claude Fable 5) --------- Co-authored-by: Claude Fable 5 <noreply@anthropic.com>
aglinxinyuan
pushed a commit
to aglinxinyuan/texera
that referenced
this pull request
Jul 5, 2026
…he head (apache#6120) ### What changes were proposed in this PR? `direct-backport-push.yml` resolved exactly one PR per push event — from `head_commit.message`, falling back to `commits/{context.sha}/pulls`. When several auto-merges land on `main` in a single ref update, only the head commit was examined and the `release/*` labels on every other PR in the same push were silently dropped: no run, no commit status, no failure comment. An audit of 2026-06-01 → 2026-07-04 found 2 of 43 labeled PRs lost this way (apache#5802, coalesced behind apache#5812; and apache#6111, coalesced behind apache#6112). Changes to the `discover` job: - Enumerate **all** commits in the push via `compareCommits(payload.before...context.sha)` (oldest first), falling back to the head commit if the compare fails or `before` is unavailable. - Resolve a PR per commit (same two strategies as before: `(#N)` title suffix, then `commits/{sha}/pulls` with backoff) and collect green `release/*` targets per PR. - Output one matrix entry per `(pr_number, merge_sha, target)` pair instead of a single PR + target list. Changes to the `push-backports` job: - Matrix is now `include: <entries>` with `max-parallel: 1`, so cherry-picks apply in commit order and two legs never race pushes to the same release branch. - `MERGE_SHA`/`PR_NUMBER` come from the matrix entry instead of `github.sha`/a single discover output; the cherry-pick, commit statuses, and PR comments all annotate the right commit and PR. - Job legs are named `backport #<pr> to <target>`; the failure-annotation job-URL matcher was updated accordingly. ### Any related issues, documentation, discussions? Closes apache#6119 ### How was this PR tested? - YAML parses (PyYAML) and all three embedded `github-script` blocks pass `node --check`. - Dry-ran the new discover logic (read-only API calls) against the two historical coalesced pushes: the 2026-07-04 push (`5c4a963f7...9cd8acf`) yields entries for **both** apache#6111 and apache#6112 with green targets, and the 2026-06-20 push range yields apache#5802 while correctly skipping the unlabeled apache#5809/apache#5811/apache#5812. The old logic produced only the head PR in each case. - Note: this fix is not retroactive — apache#5802 and apache#6111 still need manual backporting to `release/v1.2`. ### Was this PR authored or co-authored using generative AI tooling? Generated-by: Claude Code (Claude Fable 5) Co-authored-by: Claude Fable 5 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What changes were proposed in this PR?
Docs still referenced the pre-ASF
core/source tree. This PR updates every such path to the current layout:core/workflow-operator/.../edu/uci/ics/amber/operatorcommon/workflow-operator/.../org/apache/texera/amber/operatorcore/gui,/core/new-gui(operator images)frontend/src/assets/operator_imagescd core/cd core/gui(test instructions)cd frontendconfig/src/main/resources/...common/config/src/main/resources/...core/amber/.../generator_operator.py(renamed)amber/.../generator_operator_integer.pyAlso removes
docs/contribution-guidelines/micro-services-local-dev.md: it documents the now-finishedcore/micro-servicesmigration and build scripts that no longer exist; local development is covered by the developer guide andbin/local-dev.sh. No other doc links to that page.Any related issues, documentation, discussions?
Closes #6105
How was this PR tested?
Docs-only change. Every new path/link target was verified to exist in the repo:
Was this PR authored or co-authored using generative AI tooling?
Generated-by: Claude Code (Claude Fable 5)