Skip to content
This repository was archived by the owner on Sep 17, 2018. It is now read-only.

Changed Wikihouse Plugin to be compatible with Sketchup Extensions.#3

Open
MrKriss wants to merge 28 commits intotav:masterfrom
MrKriss:master
Open

Changed Wikihouse Plugin to be compatible with Sketchup Extensions.#3
MrKriss wants to merge 28 commits intotav:masterfrom
MrKriss:master

Conversation

@MrKriss
Copy link

@MrKriss MrKriss commented Mar 18, 2013

Main additions:

  • make or make build now generates wikihouse_extension.rbz
  • make release should upload following to Amazon S3 bucket:
    • wikihouse_extension.rbz and wikihouse_extension-{git-rev}.rbz
  • Included .gitignore for .rbz files
  • Renamed all instances of plugin to extension for clarity.
  • Updated install instruction in README

MrKriss added 9 commits March 14, 2013 18:02
short script to add the plugin as a SketchUp Extension. Also updated
README install instructions.
make release should upload following to Amazon S3
bucket: wikihouse_extension.rbz and wikihouse_extension-{git-rev}.rbz

Included .gitignore for .rbz files.

Renamed all instances of plugin to extension for clarity.

Updated install instruction in README
namespace recalling between different files of ruby code.

Beginning to split up wikihouse.rb into smaller files of related code. 

Run flags and config constants now in wikihouse_extension_loader before
extension is loaded.

Extra constants now in constants.rb
Utility functions and classes now in utils.rb
Output writters now in writers.rb

SVG output currently has an error. Investigating.
@MrKriss
Copy link
Author

MrKriss commented Apr 9, 2013

Hi Tav, whats the status of this? And should I continue to submit more work to this same pull request or wait till you are done with this one? I'm currently working on my own Dev branch of the folk.

@thruflo
Copy link
Collaborator

thruflo commented Apr 10, 2013

/me pokes @tav

@tav
Copy link
Owner

tav commented Apr 11, 2013

Hey @MrKriss, haven't forgotten you... just been busy with my upcoming wedding in 2 weeks! 😃

Will review and merge it soonish. As for other work you are doing, it would be great if you could break it down into individual pull requests per feature. Thanks!

check in SU 2013 that was throwing an error on loading plugin.
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.

3 participants