Skip to content
This repository has been archived by the owner on Oct 9, 2023. It is now read-only.

Add namespace to development mode plugin #14

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

zewelor
Copy link

@zewelor zewelor commented Mar 23, 2017

No description provided.

Copy link
Collaborator

@ljharb ljharb left a comment

Choose a reason for hiding this comment

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

Seems reasonable. Semver-major, though.

@@ -1,6 +1,6 @@
require 'erb'

class DevelopmentModePlugin
class Hypernova::DevelopmentModePlugin
Copy link
Collaborator

Choose a reason for hiding this comment

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

We should probably move the file into a hypernova dir to be consistent with ruby conventions.

Copy link
Author

Choose a reason for hiding this comment

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

Or add Hypernova::Plugin::DevelopmentMode and add plugin.rb file ?

Copy link
Collaborator

Choose a reason for hiding this comment

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

That seems reasonable too.

@ljharb ljharb requested a review from goatslacker March 23, 2017 17:37
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants