Skip to content
This repository has been archived by the owner on Apr 6, 2021. It is now read-only.

Create template plugin #1

Merged
merged 2 commits into from
Oct 6, 2015
Merged

Create template plugin #1

merged 2 commits into from
Oct 6, 2015

Conversation

janstenpickle
Copy link
Member

No description provided.

source 'https://rubygems.org'

gemspec
gem 'shanty', path: '/users/Chris/shanty/shanty'
Copy link
Contributor

Choose a reason for hiding this comment

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

Should remove the path from here, and instead use the Bundler config trick to make it come from local.

gem 'shanty', git: 'https://github.com/shanty/shantytown'

Now run this:

bundle config local.shanty ~/shanty/shanty

And anywhere you have shanty in a Gemfile, when you bundle exec, Bundler will use your local copy instead. Means it works for anybody who is working on this Gem as well.

provides_projects_containing "**/*.#{PROJECT_EXTENSION}"

def on_build
load_data.each do |env, data|
Copy link
Contributor

Choose a reason for hiding this comment

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

I think we should call this something other than env as this shadows the other env. Perhaps template_env?

Copy link
Member Author

Choose a reason for hiding this comment

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

oh yeah, missed that

@janstenpickle janstenpickle force-pushed the template-plugin branch 3 times, most recently from f7dcd43 to e7d2fb2 Compare October 6, 2015 08:57
end
end

RSpec.shared_context('') do
Copy link
Contributor

Choose a reason for hiding this comment

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

This is a mistake, should be removed. We're using the workspace context now.

nathankleyn added a commit that referenced this pull request Oct 6, 2015
@nathankleyn nathankleyn merged commit e9fcfc5 into master Oct 6, 2015
@nathankleyn nathankleyn deleted the template-plugin branch October 6, 2015 09:11
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.

None yet

2 participants