Skip to content

Conversation

@peyerluk
Copy link
Member

  • Add bower configuration
  • Add release grunt task

@ghost ghost assigned peyerluk Jan 24, 2014
@peyerluk
Copy link
Member Author

@matteoagosti I plan to use bower to load editable into the engine and also into the editor. So I added a bower configuration and a release task to add git version tags. The versioning is still under discussion in upfrontIO/guides#3 but I started implementing preliminary stuff so we can get a better feeling for it.

Do you like it this way? Other suggestions? Mergeable?

@matteoagosti
Copy link
Contributor

Bower makes perfectly sense for integration in the editor and the engine. So it's a go for me.
With regards to the grunt task for the release, have you been checking https://github.com/cburgdorf/grunt-release-branch ?

@peyerluk
Copy link
Member Author

Do you know if bower or npm can work with release branches?

@matteoagosti
Copy link
Contributor

Yes, with bower you just append the # to the package name, followed by the release name or the sha of the commit.

With npm you can do it for sure, by specifying the full path to the release branch. With just the package name you can specify the tag using the @ if I'm not mistaken (can't check, on the train now :p)

Il giorno 26-gen-2014, alle ore 13:26, Lukas Peyer notifications@github.com ha scritto:

Do you know if bower or npm can work with release branches?


Reply to this email directly or view it on GitHub.

@peyerluk
Copy link
Member Author

I guess you can do the same with npm. After all bower tries to copy the interface of npm as far as I know. What is the big advantage of a separate release branch?

peyerluk added a commit that referenced this pull request Jan 27, 2014
@peyerluk peyerluk merged commit 30f5557 into master Jan 27, 2014
@peyerluk peyerluk deleted the bowerify branch January 27, 2014 13:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants