Revert PR #22490 (Split PR devcontainer CI into pip and conda jobs)

[bdice]

Description

Reverts #22490 now that upstream issues breaking pip devcontainers are fixed in cuCascade and rapidsmpf.

Checklist

• I am familiar with the Contributing Guidelines.
• New or existing tests cover these changes.
• The documentation is up to date with these changes.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Enterprise

Run ID: 340954ac-aca8-480f-998f-1433a099d119

📥 Commits

Reviewing files that changed from the base of the PR and between 40de8bd and 200c077.

📒 Files selected for processing (1)

• .github/workflows/pr.yaml

🚧 Files skipped from review as they are similar to previous changes (1)

• .github/workflows/pr.yaml

---

📝 Walkthrough

Summary by CodeRabbit

• Chores
  • Consolidated CI workflow jobs, removing duplicate environment variants and simplifying the development-container build steps into a single streamlined job.
  • Cleaned up CI checks to exclude obsolete jobs and simplified the build command configuration.

---

Note: No user-facing changes; updates are limited to internal development infrastructure.

Walkthrough

GitHub workflow consolidation replaces separate devcontainers-conda and devcontainers-pip jobs with a single unified devcontainer job. Job dependencies in pr-builder, checks configuration, and build command structure are updated to reflect the consolidated workflow.

Changes

Devcontainer Job Consolidation

Layer / File(s)	Summary
Consolidated devcontainer job definition .github/workflows/pr.yaml	Job id devcontainer replaces devcontainers-conda and devcontainers-pip, with job permissions, removal of the hardcoded python_package_manager value, and an inline multi-line build_command script combining sccache setup, clone-rapidsmpf, environment configuration, and build-all steps.
Workflow dependency and checks updates .github/workflows/pr.yaml	pr-builder.needs now depends on devcontainer instead of devcontainers-conda, and checks.with.ignored_pr_jobs drops devcontainers-pip.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Possibly related PRs

• rapidsai/cudf#22490: Both PRs modify .github/workflows/pr.yaml’s devcontainer CI wiring—Split PR devcontainer CI into pip and conda jobs #22490 splits into devcontainers-conda/devcontainers-pip, while this PR consolidates them back into a single devcontainer job and updates pr-builder.needs/checks.ignored_pr_jobs plus the shared build_command accordingly.

Suggested labels

non-breaking

Suggested reviewers

• gforsyth
• KyleFromNVIDIA
• vyasr

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title clearly describes a revert of PR #22490 and directly relates to the changeset, which consolidates split devcontainer jobs back into a single job.
Description check	✅ Passed	The description explains the rationale for the revert (upstream issues fixed in cuCascade and rapidsmpf) and is directly related to the changeset modifications.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests

---

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[KyleFromNVIDIA]

/merge