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

clean up headers #5

Merged
merged 3 commits into from
Sep 30, 2021
Merged

clean up headers #5

merged 3 commits into from
Sep 30, 2021

Conversation

dzakki
Copy link
Contributor

@dzakki dzakki commented Sep 30, 2021

No description provided.

@mithi
Copy link
Owner

mithi commented Sep 30, 2021

Hi! I prefer to keep the numbering (IE 1 , 1.1, 1.2.1 ) I hope that's okay with you. Thanks 😊

@dzakki
Copy link
Contributor Author

dzakki commented Sep 30, 2021

Hi! I prefer to keep the numbering (IE 1 , 1.1, 1.2.1 ) I hope that's okay with you. Thanks 😊

Well, i will change it soon.

@mithi
Copy link
Owner

mithi commented Sep 30, 2021

Thanks 😊 Appreciate the PR!

@mithi mithi merged commit 1c53423 into mithi:main Sep 30, 2021
Rickybrow
Rickybrow reviewed Sep 30, 2021
View reviewed changes
Copy link

@Rickybrow Rickybrow left a comment

Choose a reason for hiding this comment

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

make you confident that you didn't break anything, then they didn't do their (one and only) job
For the front-end, you don't need 100%

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Rickybrow Rickybrow Rickybrow left review comments

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@dzakki @mithi @Rickybrow