Skip to content

Comments

chore: setup mise#64

Merged
nameless-mc merged 3 commits intomasterfrom
chore/setup-mise
Nov 13, 2025
Merged

chore: setup mise#64
nameless-mc merged 3 commits intomasterfrom
chore/setup-mise

Conversation

@nameless-mc
Copy link
Contributor

Why

What

How to test

Checklist

  • Read CONTRIBUTING.md at the repository.
  • Updated documentation if it is required.
  • Added/updated tests if it is required. (or tested manually)

@nameless-mc nameless-mc self-assigned this Nov 12, 2025
Copilot AI review requested due to automatic review settings November 12, 2025 04:51
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR sets up mise (a development environment management tool) for the project by adding a .mise.toml configuration file. The configuration specifies Java Corretto 17 as the project's Java version and includes a comment noting that JAVA_HOME will be automatically configured by mise.

  • Adds .mise.toml configuration file for development environment management
  • Configures Java Corretto 17 as the project's Java runtime

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

.mise.toml Outdated
# JAVA_HOME は mise が自動設定

[tools]
java = 'corretto-17'
Copy link
Contributor

Choose a reason for hiding this comment

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

@nameless-mc
Java8〜サポートなのでcorretto-8 が無難かな、と思いましたがどうでしょう
https://github.com/kintone/kintone-java-client/blob/master/README.md?plain=1#L26

Copy link
Contributor Author

Choose a reason for hiding this comment

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

その方が良いですね
3f7aaaaで対応しました

@nameless-mc nameless-mc merged commit 4e4525a into master Nov 13, 2025
2 checks passed
@nameless-mc nameless-mc deleted the chore/setup-mise branch November 13, 2025 02:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants