-
Notifications
You must be signed in to change notification settings - Fork 42
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
Fix typos in history.apt and faq.fml #208
Conversation
Fix typo/grammar.
Fix typo in faq.fml
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please rebase
@michael-o sorry to ask so bluntly - do you mean I am supposed to squash my commits or what do you mean by rebasing? As there seem no conflicts I do not really know what I am supposed to do. Sry |
@michael-o sorry I synced my fork and hope everything can be merged now. Sry |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fast forward only, no intermediate commits
@michael-o I have no configuration option when syncing my fork ..... I thought you would get rid of my 5 commits during merging my PR? Where is my misunderstanding of your request ....? |
Will try to open a new PR - sry4the mess. |
Trivial changes after reading the site. Thanks