Skip to content
This repository has been archived by the owner on Oct 3, 2024. It is now read-only.

fixed rultor merging script #26

Merged
merged 1 commit into from
Jun 5, 2023
Merged

Conversation

Graur
Copy link
Contributor

@Graur Graur commented Jun 5, 2023

PR-Codex overview

This PR aims to improve the build process by enabling LC_ALL=C in the merge process to avoid issues with non-ASCII characters. It also removes unnecessary environment variables.

Detailed summary

  • Added a todo to enable LC_ALL=C in the merge process
  • Removed JAVA_TOOL_OPTIONS and MAVEN_OPTS environment variables from the merge and release processes
  • Removed LC_ALL=C from the merge script
  • Updated the release script to set the new version and commit it to Git

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

@Graur
Copy link
Contributor Author

Graur commented Jun 5, 2023

@yegor256 I fixed rultor merging script which should allow us to merge pull requests in this repository. Can you please check it and merge it manually?
Or we can merge #25 manually if you think it would be better

@yegor256 yegor256 merged commit b42a66b into objectionary:master Jun 5, 2023
@yegor256
Copy link
Member

yegor256 commented Jun 5, 2023

@Graur done

@Graur Graur deleted the fixed-rultor.yml branch June 5, 2023 11:13
@Graur
Copy link
Contributor Author

Graur commented Jun 5, 2023

@yegor256 Thanks!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants