Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

CHANGELOG parser by date #426

Merged
merged 20 commits into from
Jul 5, 2023
Merged

CHANGELOG parser by date #426

merged 20 commits into from
Jul 5, 2023

Conversation

kateliu20
Copy link
Contributor

This parses the CHANGELOG.md file by date, and only shows the updated file if the latest entry does not match the previous entry by date.

return LOCAL_DATE_REGEX.matches(this)
}

object ChangelogParser {
Copy link
Collaborator

Choose a reason for hiding this comment

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

Let's add a doc to this

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Doc added: 37f1b00

Copy link
Collaborator

Choose a reason for hiding this comment

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

I think it landed in the wrong place? I don't see it on ChangelogParser

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I added a doc to ChangelogParser.kt

@kateliu20 kateliu20 added the skate 🛹 Anything related to the Skate IntelliJ plugin label Jun 28, 2023
@kateliu20 kateliu20 requested a review from arpita184 June 29, 2023 00:33
@github-actions

This comment was marked as off-topic.

@kateliu20 kateliu20 requested a review from arpita184 June 30, 2023 22:28
return LOCAL_DATE_REGEX.matches(this)
}

object ChangelogParser {
Copy link
Collaborator

Choose a reason for hiding this comment

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

I think it landed in the wrong place? I don't see it on ChangelogParser

@kateliu20 kateliu20 added this pull request to the merge queue Jul 5, 2023
Merged via the queue into main with commit 2510066 Jul 5, 2023
@kateliu20 kateliu20 deleted the kl/change_log_parser branch July 5, 2023 19:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
skate 🛹 Anything related to the Skate IntelliJ plugin
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants