Skip to content
This repository has been archived by the owner on Nov 9, 2022. It is now read-only.

Adding roxy property to ignore certain files or folders #331

Closed

Conversation

miguelrgonzalez
Copy link
Contributor

Specially when working with node and bower there are certain folder we don't want to be deployed into the server.

@grtjn
Copy link
Contributor

grtjn commented Jan 12, 2015

Have you considered keeping those outside src/?

@miguelrgonzalez
Copy link
Contributor Author

Yes, I did.
But I prefer to have source code inside the src folder.
Plus having bower.js, gulp.js there also makes sense.

@@ -204,3 +204,6 @@ mlcp-vmargs=-Xmx512m
# Temporary fix to support MarkLogic 8 EA2
#
evaler-port=7998

# folde to ignore when deploying into modules database
ignore-folders=.*/src/(node_modules|.tmp)/.*
Copy link
Contributor

Choose a reason for hiding this comment

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

Move this into build.sample.properties as a comment explaining how to use it. I don't think this should be the default for everyone.

@paxtonhare
Copy link
Contributor

@miguelrgonzalez I like this and think it's useful. I added some inline comments that I'd like to see fixed before we merge it.

@miguelrgonzalez
Copy link
Contributor Author

Hi Paxton,
will do the requested changes and fix my commit.

Miguel Rodriguez added 2 commits January 12, 2015 16:19
@paxtonhare
Copy link
Contributor

Awesome. Thanks for submitting it.

@dmcassel
Copy link
Collaborator

+1 on this, once Paxton's comments are addressed

@grtjn
Copy link
Contributor

grtjn commented Jan 14, 2015

One sec before merging. I think it would be better to push this into the dev branch.

@miguelrgonzalez would it be a lot of trouble to ask you to redo your changes based on the dev branch, and do a PR against that one?

@dmcassel
Copy link
Collaborator

Good catch, @grtjn

@miguelrgonzalez
Copy link
Contributor Author

No problem, it's just a couple of lines really.

@grtjn
Copy link
Contributor

grtjn commented Jan 16, 2015

Replaced by PR #337..

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.

4 participants