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

few minor changes #60

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

few minor changes #60

wants to merge 3 commits into from

Conversation

ghost
Copy link

@ghost ghost commented Jul 9, 2020

New base map added and some grammar tweaks

nvgoldin
nvgoldin requested changes Jul 9, 2020
View reviewed changes
Copy link
Contributor

@nvgoldin nvgoldin left a comment

Choose a reason for hiding this comment

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

Hey @rom-fruitspec - thanks for the contribution!

I see loads of changes in the files - they mostly look like formatting changes. Could you pinpoint to the logical changes you've made? (just send a link to the line of code changes)

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

@ArielBatkilin ArielBatkilin Awaiting requested review from ArielBatkilin

@jherskow jherskow Awaiting requested review from jherskow jherskow is a code owner

@matanel-oren matanel-oren Awaiting requested review from matanel-oren matanel-oren is a code owner

@Nitay880 Nitay880 Awaiting requested review from Nitay880 Nitay880 is a code owner

Requested changes must be addressed to merge this pull request.

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

Successfully merging this pull request may close these issues.
2 participants
@nvgoldin @romzats