Skip to content

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

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 😄

.eslintrc.json Outdated
{
"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.

4 participants