Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Create different builds of HTML5 Boilerplate? #1625

Closed
wants to merge 1 commit into from
Closed

Conversation

alrra
Copy link
Member

@alrra alrra commented Oct 27, 2014

There was a discussion in the first part of the year about removing the .htaccess file from the project.

I personally think that including some boilerplate server configurations is important, but I do agree that users should have a choice in the matter! So, with this pull request I propose that we slowly move towards providing different builds of HTML5 Boilerplate.

Right now, the changes from this PR will generate two versions:

  • dist/h5bp (no server configs)
  • dist/h5bp+apache (Apache server configs)

and also two archives that we can attache to a release:

But in the near future (after v5) my hope is that we'll provide also a nginx build (already talked with @AD7six about it) and so on.

Thoughts? (Cc: @h5bp/html5-boilerplate)

Also, any other ideas are welcomed!

The changes from this commit will make the build script create
two different versions of HTML5 Boilerplate, one with the Apache
server configurations and one without.

The reason behind this change is to allow¹ users to choose if they
want the Apache server configs or not, and to move slowly towards
providing multiple builds of HTML5 Boilerplate that contain different
server configs.

- - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - -

¹ #1527
@paulirish
Copy link
Member

shipping .htaccess by default sets up a good pattern of delivering performance by default. Every noob that doesn't know better gets to ship this to their hosting environment getting gzip/expires out of the box.

I would prefer to ship it by default and create a "naked" distribution without it.

@alrra
Copy link
Member Author

alrra commented Oct 27, 2014

I would prefer to ship it by default and create a "naked" distribution without it.

On the site we can still offer the Apache build by default, but we can also link to the release page where users can download a different build (and hopefully if the future we'll have more than two).

@QWp6t
Copy link
Contributor

QWp6t commented Oct 27, 2014

Another possible build option would be one that doesn't have analytics, since there were a number of folks who strongly disagreed with including GA.

@patrickkettner
Copy link
Contributor

What about a builder with options, rather than static builds?
On Oct 27, 2014 5:05 PM, "QWp6t" notifications@github.com wrote:

Another possible build option would be one that doesn't have analytics,
since there were a number of folks who strongly disagreed with including GA.


Reply to this email directly or view it on GitHub
#1625 (comment)
.

@alrra
Copy link
Member Author

alrra commented Oct 27, 2014

Another possible build option would be one that doesn't have analytics, since there were a number of folks who strongly disagreed with including GA.

Going on that route will end up basically having a site builder or something. I've proposed¹ this due to the fact that server configs are usually harder to deal with then just removing a couple of lines (e.g.: .htaccess is invisible on some systems, so some users won't know it's there or how to delete it)

¹ that doesn't mean we'll necessarily do it

@alrra
Copy link
Member Author

alrra commented Oct 27, 2014

Another idea is to make the yeoman generator more useful. However since it is a command line tool, it won't be that useful for quite a good portion of the users.

@arthurvr
Copy link
Member

Not hugely important, but once this is merged this piece of the docs should be updated.

I would prefer to ship it by default and create a "naked" distribution without it.

Definitely. Some just getting started designers or front-end developers might not know anything about servers, and for those people it's better to include all those smart defaults in the .htaccess file.

But in the near future (after v5) my hope is that we'll provide also a nginx build (already talked with @AD7six about it) and so on.

Is this difficult to integrate? If not, we could directly include builds for all other server configs.

Another idea is to make the yeoman generator more useful.

+1. Right now we aren't using enough of yeoman' power.

@alrra alrra changed the title Create different builds of HTML5 Boilerplate Create different builds of HTML5 Boilerplate? Oct 27, 2014
@AD7six
Copy link
Member

AD7six commented Oct 28, 2014

Is this difficult to integrate?

I don't think so - it's just creating a single archive with naked-fronted template + server configs.

Regarding bundling frontend+server in general: To me it depends what users want/would find useful. I think it's important to not take two separate projects which people use, and bundle them together creating something that a few people would use. Sometimes the same person uses both, and other times they are most definitely the concern of different people/roles.

@alrra
Copy link
Member Author

alrra commented Oct 28, 2014

To me it depends what users want/would find useful.

I gave this some more thought and I think/agree that before embarking on a change like this it would be useful to ask a wider range of people (even the ones that aren't on GitHub) what they use and how they use it.

Closing, will open a different issue regarding this after v5 is out.

@alrra alrra closed this Oct 28, 2014
@alrra alrra deleted the multiple_builds branch October 28, 2014 11:20
@ghost
Copy link

ghost commented Oct 28, 2014

Why deleted the branch?

@alrra

It is something we discuss later, no?

@alrra
Copy link
Member Author

alrra commented Oct 28, 2014

Why deleted the branch?

@emoju Because for the time being it is not needed, and if for some reason we'll ever need the code, the branch can be easily restored.

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.

6 participants