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

Add examples to rewrite docs #608

Merged
merged 4 commits into from
Dec 4, 2024
Merged

Add examples to rewrite docs #608

merged 4 commits into from
Dec 4, 2024

Conversation

eyihluyc
Copy link
Member

@eyihluyc eyihluyc commented Dec 4, 2024

Addresses #606

@eyihluyc eyihluyc requested a review from deemp December 4, 2024 20:55
### `--chain` `--tex`

```$ as tex
eo-phi-normalizer rewrite --chain --tex bar.phi
Copy link
Member

Choose a reason for hiding this comment

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

There's no bar.phi in the repo. I'll add generation of this file.

@deemp deemp self-requested a review December 4, 2024 23:33
@deemp deemp merged commit ba1002c into master Dec 4, 2024
8 checks passed
@0crat
Copy link

0crat commented Dec 6, 2024

@eyihluyc Hey there! 👋 Great job on your contribution! You've earned +4 points, which is the minimum we give. Here's the breakdown: +4 base points, -4 because there was only 1 comment (we like to see more engagement!), and +4 to keep you motivated. Remember, more hits-of-code and active code reviews can boost your points next time! Keep the contributions coming and aim for that 24-point max. 😉 Your current balance is +4. Looking forward to your next awesome contribution!

@0crat
Copy link

0crat commented Dec 6, 2024

@deemp Hey there! 👋 Thanks for your review! You've snagged +4 points this time. Here's the breakdown: +4 base points, -24 for self-review (oops!), +0.01 per hit-of-code (capped at 16), -10 for having less than 6 comments, and +0.25 per comment (max 8). We adjusted it to the minimum 4 points to keep you motivated! Your running balance is now +52. Keep those reviews coming, and remember, more comments could boost your points next time! 💪

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.

3 participants