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

Sync settings after installing the integration #72

Closed
wants to merge 7 commits into from
Closed

Sync settings after installing the integration #72

wants to merge 7 commits into from

Conversation

knrt10
Copy link

@knrt10 knrt10 commented Mar 17, 2018

This is reference to issue #1

Added Functionalities:

When a user has a predefined .github folder and they install the probot/settings app after that, then 2 functionalities are added

1.) If .github folder has settings.yml file then it will be automatically synced.

2.) If no settings.yml file is found in .github then the bot opens a PR with some default settings in settings.yml file.

@ci-reporter
Copy link

ci-reporter bot commented Mar 17, 2018

The build is failing

✨ Good work on this PR so far! ✨ Unfortunately, the Travis CI build is failing as of fc15878. Here's the output:

npm test
> probot-settings@1.0.0 test /home/travis/build/probot/settings
> jest && standard

 PASS  test/lib/plugins/repository.test.js
 PASS  test/lib/plugins/collaborators.test.js
 PASS  test/lib/plugins/labels.test.js
 PASS  test/lib/plugins/teams.test.js
 PASS  test/index.test.js

Test Suites: 5 passed, 5 total
Tests:       9 passed, 9 total
Snapshots:   0 total
Time:        6.036s
Ran all test suites.
standard: Use JavaScript Standard Style (https://standardjs.com)
standard: Run `standard --fix` to automatically fix some problems.
  /home/travis/build/probot/settings/index.js:30:7: Extra space after key 'owner'.
  /home/travis/build/probot/settings/index.js:31:7: Extra space after key 'repo'.
  /home/travis/build/probot/settings/index.js:33:1: Expected indentation of 4 spaces but found 6.

I'm sure you can fix it! If you need help, don't hesitate to ask a maintainer of the project!


This comment was automagically generated by ci-reporter. If you see a problem, open an issue here.

@knrt10 knrt10 closed this Mar 19, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant