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

Maintenance #3

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

Maintenance #3

wants to merge 21 commits into from

Conversation

steelbrain
Copy link
Member

  • Add CI config
  • Tweak ignore files
  • interface -> decls
  • Tweak flow config
  • Add ESLint
  • Update license
  • Update code to new ESLint config

@@ -1,6 +1,6 @@
MIT License

Copyright (c) 2016 Philipp von Radziewsky
Copyright (c) 2016 AtomDebugger Team (Philipp von Radziewsky & Steel Brain)
Copy link
Member Author

Choose a reason for hiding this comment

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

@steelbrain
Copy link
Member Author

@philippvr Ping me on Atom Slack when you're on, got a few things to discuss about the codebase

export default {
instance: null,
activate() {
require('atom-package-deps').install('debugger') // eslint-disable-line
Copy link
Member

Choose a reason for hiding this comment

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

The package shouldn't depend on itself.

Copy link
Member Author

Choose a reason for hiding this comment

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

Well the package is not depending on itself 😉
It's just telling atom-package-deps the name of the current package to install dependencies of

Copy link
Member Author

Choose a reason for hiding this comment

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

Copy link
Member

Choose a reason for hiding this comment

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

OK, I should check before making comments. 😊

Copy link
Member Author

Choose a reason for hiding this comment

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

I appreciate the keen eye 😄

{
"extends": "steelbrain",
"rules": {
"no-param-reassign": 0,

Choose a reason for hiding this comment

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

0 -> "off" 😛

@gsmcmullin gsmcmullin mentioned this pull request Nov 21, 2016
7 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants