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.

Sign up for GitHub

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

Jump to bottom

Format code using cljfmt #102

Merged
merged 1 commit into from
Oct 2, 2024
Merged

Format code using cljfmt #102

merged 1 commit into from
Oct 2, 2024

Conversation

crisptrutski
Copy link
Contributor

@crisptrutski crisptrutski commented Oct 2, 2024 •
edited
Loading

🧹

@crisptrutski crisptrutski mentioned this pull request Oct 2, 2024
Merged
@crisptrutski Graphite App
Copy link
Contributor Author

crisptrutski commented Oct 2, 2024 •
edited
Loading

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @crisptrutski and the rest of your teammates on Graphite Graphite

@crisptrutski crisptrutski mentioned this pull request Oct 2, 2024
Merged
@crisptrutski crisptrutski changed the title Format files Format code using cljfmt Oct 2, 2024
@crisptrutski crisptrutski marked this pull request as ready for review October 2, 2024 06:38
@crisptrutski crisptrutski requested a review from a team October 2, 2024 06:38
@crisptrutski crisptrutski self-assigned this Oct 2, 2024
crisptrutski
crisptrutski commented Oct 2, 2024
View reviewed changes
test/macaw/core_test.clj
(dotimes [_ 1000] (simple-benchmark)))
(dotimes [_ 1000] (simple-benchmark)))
Copy link
Contributor Author

@crisptrutski crisptrutski Oct 2, 2024 •
edited
Loading

Choose a reason for hiding this comment

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

It can't tell that this is under a macro, because we use a dynamic require. Doesn't feel worth overriding.

@crisptrutski crisptrutski changed the base branch from more-acceptance-tests to graphite-base/102 October 2, 2024 15:10
@crisptrutski crisptrutski changed the base branch from graphite-base/102 to master October 2, 2024 15:11
@crisptrutski crisptrutski merged commit 828d90b into master Oct 2, 2024
5 checks passed
@crisptrutski Graphite App
Copy link
Contributor Author

Merge activity

@crisptrutski crisptrutski deleted the cljfmt branch October 2, 2024 15:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@tsmacdonald tsmacdonald tsmacdonald approved these changes

@qnkhuat qnkhuat qnkhuat approved these changes

Assignees

@crisptrutski crisptrutski

Labels
.Team/BackendComponents
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@crisptrutski @tsmacdonald @qnkhuat