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.

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

reports: Add anctartica's GSoC report #630

Merged
merged 1 commit into from
Aug 13, 2018
Merged

Conversation

saksham189
Copy link
Contributor

@saksham189 saksham189 commented Aug 11, 2018

[Muhammad Kaisar Arkhan](https://github.com/yukiisbored) & [Matt Bodenhamer]
(https://github.com/mbodenhamer) & [Andrey Safonov](https://github.com/xferra)
phase:
- Bonding : https://gitlab.com/coala/GSoC/GSoC-2018/milestones/10
Copy link
Member

Choose a reason for hiding this comment

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

see #627 and #615

- repo: bear-install-playbook
- link: https://gitlab.com/coala/bear-install-playbook/commit/c82a9737c6852d09f681a7fc98a0f094dd38194b
- details: >
Run coala static code analysis
Copy link
Member

Choose a reason for hiding this comment

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

this was an automated change? if so, discard

Copy link
Contributor Author

Choose a reason for hiding this comment

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

removed all the automated changes :)

@saksham189 saksham189 force-pushed the report branch 2 times, most recently from 79fb79a to 667fe34 Compare August 11, 2018 17:19
[Matt Bodenhamer](https://github.com/mbodenhamer),
[Andrey Safonov](https://github.com/xferra)
phase:
- Bonding : https://gitlab.com/coala/GSoC/GSoC-2018/milestones/10
Copy link
Member

Choose a reason for hiding this comment

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

#630 (comment) not done

@jayvdb jayvdb changed the title reports: Add anctaritca's GSoC report reports: Add anctartica's GSoC report Aug 12, 2018
[Matt Bodenhamer](https://github.com/mbodenhamer),
[Andrey Safonov](https://github.com/xferra)
phase:
- Bonding: https://gitlab.com/coala/GSoC/GSoC-2018/milestones/10
Copy link
Member

Choose a reason for hiding this comment

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

indentation; use yamllint

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I don't get anything from yamllint on this line

@TravisBuddy
Copy link

Travis tests have failed

Hey @anctartica,
Please read the following log in order to understand the failure reason.
It'll be awesome if you fix what's wrong and commit the changes.

Ruby: 2.5.1

docker run -v=$(pwd):/app --workdir=/app coala/base coala --ci
Unable to find image 'coala/base:latest' locally
latest: Pulling from coala/base
















Digest: sha256:125d6cf24929d39db47b3ccd83d6ed1b6a231fc7e8afbf70bc4b5bd87653b0ff
Status: Downloaded newer image for coala/base:latest
Executing section global...
[WARNING][13:59:27] InvalidLinkBear: This result has no patch attached.

_reports/li-boxuan.md
| 341| coala·gh-board·[website](https://coala-gh-board.netlify.com/)
|    | [MAJOR] InvalidLinkBear:
|    | Broken link - unable to connect to https://coala-gh-board.netlify.com/
Executing section commit...
Executing section linecount...
Executing section filenames...
Executing section spacing...
Executing section html...
Executing section yml...
Executing section new_linelength_projects...
Executing section old_linelength_projects...
Executing section cli...

@@ -0,0 +1,266 @@
---
layout: report
permalink: /GSoC/2018/StatusReport/SakshamBansal
Copy link
Member

Choose a reason for hiding this comment

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

see updates to the example. needs date, categories, and not layout or permalink, etc.

tarball: https://rawgit.com/anctartica/GSoC-Report/master/work.tar.gz
sha256sum: f413963583c7e18a5ccb42f21405290edcf79a0f8c61106ff21671efe0b98238
mentors: >
[Muhammad Kaisar Arkhan](https://github.com/yukiisbored),
Copy link
Member

Choose a reason for hiding this comment

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

yamllint.

activity:
- 1:
- repo: cEPs
- link: https://github.com/coala/cEPs/commit/ca27c58eef778fd7b3aff3b874b0e58f56e4a559
Copy link
Member

Choose a reason for hiding this comment

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

you can trim these to 8 character shas.
up to you

organisation_link: https://coala.io
project: Improve installation of coala and coala-bears
project_link: https://summerofcode.withgoogle.com/projects/#4642118669172736
tarball: https://rawgit.com/anctartica/GSoC-Report/master/work.tar.gz
Copy link
Member

Choose a reason for hiding this comment

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

extracting this I get

tar: Ignoring unknown extended header keyword 'LIBARCHIVE.creationtime'
tar: Ignoring unknown extended header keyword 'SCHILY.dev'
tar: Ignoring unknown extended header keyword 'SCHILY.ino'
tar: Ignoring unknown extended header keyword 'SCHILY.nlink'
tar: Ignoring unknown extended header keyword 'SCHILY.dev'
tar: Ignoring unknown extended header keyword 'SCHILY.ino'
tar: Ignoring unknown extended header keyword 'SCHILY.nlink'

Copy link
Member

Choose a reason for hiding this comment

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

Also contains .DS_Store

Copy link
Contributor Author

Choose a reason for hiding this comment

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

This was due to BSD tar (on mac) vs. GNU tar.
Should be fixed now

Copy link
Member

Choose a reason for hiding this comment

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

Still has one .DS_Store in it, but good enough

- link: https://gitlab.com/coala/bear-install-playbook/commit/7dee7591927c0883269f5741102b90bfb93f84ff
- details: >
Add instructions on using playbook
- 35:
Copy link
Member

Choose a reason for hiding this comment

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

59 patches vs 35 work activity entries.

I see some patches were ust sync' but I easily see some real work patches which are not included here

Copy link
Contributor Author

Choose a reason for hiding this comment

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

the others were really very small to include

Copy link
Contributor Author

Choose a reason for hiding this comment

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

alright added some more

Copy link
Member

Choose a reason for hiding this comment

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

Small is ok, as long as it isnt automated or silly.

organisation_link: https://coala.io
project: Improve installation of coala and coala-bears
project_link: https://summerofcode.withgoogle.com/projects/#4642118669172736
tarball: https://rawgit.com/anctartica/GSoC-Report/master/work.tar.gz
Copy link
Member

Choose a reason for hiding this comment

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

Still has one .DS_Store in it, but good enough

@jayvdb
Copy link
Member

jayvdb commented Aug 13, 2018

ack 6453618

@jayvdb
Copy link
Member

jayvdb commented Aug 13, 2018

@gitmate-bot ff

@gitmate-bot
Copy link

Hey! I'm GitMate.io! This pull request is being fastforwarded automatically. Please DO NOT push while fastforward is in progress or your changes would be lost permanently ⚠️

@gitmate-bot
Copy link

Automated fastforward with GitMate.io was successful! 🎉

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

Successfully merging this pull request may close these issues.

4 participants