Add main.min.css and main.min.js to gitignore

[ATofighi]

We should add public/assets/css/main.min.css, public/assets/css/rtl.min.css, public/assets/js/main.js.min.map, public/assets/js/vendor.js.min.map, public/assets/js/main.min.js and public/assets/js/vendor.min.js to .gitignore...
these files make merge conflicts usually and we don't use them now... (https://github.com/mybb/mybb2/blob/master/resources/views/footer/footer.twig#L17-L18 & https://github.com/mybb/mybb2/blob/master/resources/views/partials/headerinclude.twig#L7,L9 )

[wpillar]

👍 no compiled files should be committed to the repository imho.

[euantorano]

The reason they were committed was so that not everybody would need node, bower, gulp etc set up.

On 30 May 2015, at 12:07, Will Pillar [email protected] wrote:

no compiled files should be committed to the repository imho.

—
Reply to this email directly or view it on GitHub.

[ATofighi]

We use main.js and main.css and vendor.js now, minified files can make merge conflicts for now....

[wpillar]

@euantorano when you say 'everybody' do you mean developers or users?

[euantorano]

I mean developers and SQA team members. Not everybody needs to be able to do work on front end stuff.

On 30 May 2015, at 13:00, Will Pillar [email protected] wrote:

@euantorano when you say 'everybody' do you mean developers or users?

—
Reply to this email directly or view it on GitHub.

[wpillar]

If we set it up properly, it would be no more difficult than:

git checkout <branch_name>
make build

Done. I'd rather that than the conflicts we have now to be honest.

[euantorano]

True.

On 30 May 2015, at 13:16, Will Pillar [email protected] wrote:

If we set it up properly, it would be no more difficult than:

git checkout <branch_name>
make build
Done. I'd rather that than the conflicts we have now to be honest.

—
Reply to this email directly or view it on GitHub.