Skip to content

Conversation

@raulcd
Copy link
Member

@raulcd raulcd commented Dec 4, 2024

Rationale for this change

We are not using JIRA anymore in the project

What changes are included in this PR?

Remove JIRA from:

  • documentation
  • pr and issue templates
  • comment bot
  • merge script

Are these changes tested?

Via CI

Are there any user-facing changes?

No

@github-actions
Copy link

github-actions bot commented Dec 4, 2024

⚠️ GitHub issue #44934 has been automatically assigned in GitHub to PR creator.

@raulcd
Copy link
Member Author

raulcd commented Dec 4, 2024

@kou @pitrou this only contains the developer tools updates (merge script, bot comments, pr templates and some minor docs). It does not contain the archery required changes but it's already slightly big. I think it would be better to split the archery changes on a different PR. If you are ok with that I'll create a new issue and would update descriptions accordingly.

@raulcd raulcd marked this pull request as ready for review December 4, 2024 15:28
@pitrou pitrou changed the title GH-44934: [Dev][Archery] Remove JIRA remnants GH-44934: [Dev] Remove JIRA remnants Dec 4, 2024
@pitrou
Copy link
Member

pitrou commented Dec 4, 2024

Did you forget to update https://github.com/apache/arrow/blob/main/dev/requirements_merge_arrow_pr.txt ?

@raulcd
Copy link
Member Author

raulcd commented Dec 4, 2024

Copy link
Member

@kou kou left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

+1

Let's work on Archery as a separated task.

@github-actions github-actions bot added awaiting merge Awaiting merge and removed awaiting committer review Awaiting committer review labels Dec 5, 2024
Co-authored-by: Sutou Kouhei <kou@cozmixng.org>
@raulcd
Copy link
Member Author

raulcd commented Dec 5, 2024

I've created this issue to work on the archery changes:

@github-actions github-actions bot added awaiting changes Awaiting changes and removed awaiting merge Awaiting merge labels Dec 5, 2024
@github-actions github-actions bot added awaiting change review Awaiting change review and removed awaiting changes Awaiting changes labels Dec 5, 2024
@assignUser assignUser requested a review from thisisnic as a code owner December 5, 2024 23:28
@assignUser assignUser merged commit fe32a7d into apache:main Dec 5, 2024
9 checks passed
@assignUser assignUser removed the awaiting change review Awaiting change review label Dec 5, 2024
@assignUser
Copy link
Member

@amoeba can you remove the mentions of jira in the release guide when you update it?

@amoeba
Copy link
Member

amoeba commented Dec 5, 2024

Yep, I'll plan to get that release guide update in for the 19.0.0 release.

@raulcd raulcd deleted the GH-44934 branch December 5, 2024 23:47
@conbench-apache-arrow
Copy link

After merging your PR, Conbench analyzed the 3 benchmarking runs that have been run so far on merge-commit fe32a7d.

There were 132 benchmark results with an error:

There were no benchmark performance regressions. 🎉

The full Conbench report has more details. It also includes information about 20 possible false positives for unstable benchmarks that are known to sometimes produce them.

kou added a commit to kou/arrow that referenced this pull request Dec 9, 2024
pitrou pushed a commit that referenced this pull request Dec 9, 2024
### Rationale for this change

It's removed by #44936.

### What changes are included in this PR?

Remove `getJiraInfo` from `exports`.

### Are these changes tested?

No. But this will resolve the reported error.

### Are there any user-facing changes?

No.

Authored-by: Sutou Kouhei <kou@clear-code.com>
Signed-off-by: Antoine Pitrou <antoine@python.org>
pitrou pushed a commit to pitrou/arrow that referenced this pull request Dec 10, 2024
We are not using JIRA anymore in the project

Remove JIRA from:
- documentation
- pr and issue templates
- comment bot
- merge script

Via CI

No
* GitHub Issue: apache#44934

Lead-authored-by: Raúl Cumplido <raulcumplido@gmail.com>
Co-authored-by: Jacob Wujciak-Jens <jacob@wujciak.de>
Co-authored-by: Sutou Kouhei <kou@cozmixng.org>
Signed-off-by: Jacob Wujciak-Jens <jacob@wujciak.de>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants