-
Notifications
You must be signed in to change notification settings - Fork 53
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: minor, fix some typos in README. #633
base: master
Are you sure you want to change the base?
Conversation
@@ -20,7 +20,7 @@ jobs: | |||
# | |||
- run: sudo sed -i 's;http://archive.debian.org/debian/;http://deb.debian.org/debian/;' /etc/apt/sources.list | |||
- run: sudo apt-get -y update | |||
- run: sudo apt-get -y install make autopoint texinfo libtool intltool bzip2 gettext clang-3.5 clang-4.0 g++-multilib | |||
- run: sudo apt-get -y --force-yes install make autopoint texinfo libtool intltool bzip2 gettext clang-3.5 clang-4.0 g++-multilib |
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.
I added the --force-yes
here, because without it, this line would fail with an error message saying:
WARNING: The following packages cannot be authenticated!
...list of various packages...
E: There are problems and -y was used without --force-yes
See here for the error. The same error also comes up for other PRs, and is the reason that CI is red for them.
@kevina Could you have a look at this? |
Hi @dfsp-spirit: Sorry for the very long delay. CircleCI should be fixed now. Can you please rebase and squash your commits on top of master. Also please reword your commit message to start with "docs: ", for example "docs: fix some typos in README" |
Thanks for the response! I'll do it. |
This tiny PR just fixes a few typos in the README file.