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

[env] Introduce elm review #18

Merged
merged 2 commits into from
Nov 29, 2023
Merged

[env] Introduce elm review #18

merged 2 commits into from
Nov 29, 2023

Conversation

y047aka
Copy link
Owner

@y047aka y047aka commented Nov 25, 2023

elm-reviewを簡単に導入します。

Comment on lines +36 to +37
-- , NoUnused.Exports.rule
-- , NoUnused.Modules.rule
Copy link
Owner Author

Choose a reason for hiding this comment

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

将来的に使いたいけど、現時点では無効にしています。

@y047aka y047aka changed the title Introduce elm review [env] Introduce elm review Nov 25, 2023
Copy link
Collaborator

@pxfnc pxfnc left a comment

Choose a reason for hiding this comment

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

lgtmです!そろそろgithub actionでci回し始めてもよさそうな気もしてきますね

@y047aka
Copy link
Owner Author

y047aka commented Nov 29, 2023

lgtmです!そろそろgithub actionでci回し始めてもよさそうな気もしてきますね

(設定追加をサボりましたの札を下げています)

@y047aka y047aka merged commit 30a7d2f into main Nov 29, 2023
1 check passed
@y047aka y047aka deleted the introduce_elm-review branch November 29, 2023 10:48
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