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

Edit for more readability #2630

Merged
merged 4 commits into from
Nov 17, 2021
Merged

Edit for more readability #2630

merged 4 commits into from
Nov 17, 2021

Conversation

ciminbinazadeh
Copy link
Contributor

@ciminbinazadeh ciminbinazadeh commented Nov 13, 2021 •
edited
Loading

Hi
Edit for more readability.
Good luck

  • Have you followed the contributing guidelines?
  • Have you explained what your changes do, and why they add value to the Guides?

Please note: we will close your PR without comment if you do not check the boxes above and provide ALL requested information.

@ciminbinazadeh
Edit for more readability
c966e4e 
Hi
Edit for more readability.
Good luck
@ciminbinazadeh ciminbinazadeh requested a review from a team as a code owner November 13, 2021 07:06
@ciminbinazadeh
Copy link
Contributor Author

Please help me for This error "Process completed with exit code 1."

@ciminbinazadeh
Update legal.md
20d2e28 
Re-edit
@ciminbinazadeh
Update legal.md
d900d0c 
re-edit for readability
Amoon79
Amoon79 reviewed Nov 16, 2021
View reviewed changes
Copy link

@Amoon79 Amoon79 left a comment

Choose a reason for hiding this comment

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

Can't understand scriptures,, not one correct sentence

@nholden
Copy link
Contributor

nholden commented Nov 16, 2021

Please help me for This error "Process completed with exit code 1."

👋 Hi, @ciminbinazadeh! I see this annotation on the files changed tab. I think addressing it could get the tests passing.

Screen Shot 2021-11-16 at 2 12 26 PM

@ciminbinazadeh
Copy link
Contributor Author

I see this annotation on the files changed tab. I think addressing it could get the tests passing.
Hi @nholden
thank you 🙏🌸

@ciminbinazadeh
fixed error
aec3eca 
Re-edit to fix the error
@Amoon79
Copy link

Amoon79 commented Nov 17, 2021

Thanks to nick and his amazing magic work ,, glad u have it done

nholden
nholden approved these changes Nov 17, 2021
View reviewed changes
Copy link
Contributor

@nholden nholden 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 this contribution! 💖

@nholden nholden merged commit adacf38 into github:main Nov 17, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Amoon79 Amoon79 Amoon79 left review comments

@nholden nholden nholden approved these changes

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

Successfully merging this pull request may close these issues.
4 participants
@ciminbinazadeh @nholden @Amoon79 @Thefaceofasaba