Skip to content

Add comment for :erlang in captains_log#2395

Merged
jiegillet merged 2 commits intoexercism:mainfrom
FraSanga:captains-log-erlang
Feb 28, 2026
Merged

Add comment for :erlang in captains_log#2395
jiegillet merged 2 commits intoexercism:mainfrom
FraSanga:captains-log-erlang

Conversation

@FraSanga
Copy link
Contributor

Copilot AI review requested due to automatic review settings February 24, 2026 14:17
@github-actions github-actions bot added track/elixir Elixir track type/analyzer-comments Analyzer comments labels Feb 24, 2026
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

Adds a new analyzer feedback snippet for the Elixir “captains-log” exercise to guide solutions toward using Erlang’s :erlang module for date formatting, aligning with the linked discussion.

Changes:

  • Added a new analyzer comment file for recommending :erlang usage in captains-log.

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

Copy link
Contributor

@jiegillet jiegillet left a comment

Choose a reason for hiding this comment

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

Thanks for opening a PR here as well!

@jiegillet jiegillet self-requested a review February 28, 2026 02:20
Copy link
Contributor

@jiegillet jiegillet left a comment

Choose a reason for hiding this comment

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

Perfect, thank you!

@jiegillet jiegillet merged commit c4ede3a into exercism:main Feb 28, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

track/elixir Elixir track type/analyzer-comments Analyzer comments

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants