Skip to content

Remove --require-path completely#1996

Closed
paf31 wants to merge 3 commits intomasterfrom
no-require-path
Closed

Remove --require-path completely#1996
paf31 wants to merge 3 commits intomasterfrom
no-require-path

Conversation

@paf31
Copy link
Copy Markdown
Contributor

@paf31 paf31 commented Apr 5, 2016

This is the first of several PRs to remove the stuff we deprecated in 0.8.*.

@paf31 paf31 added this to the 0.9.0 milestone Apr 5, 2016
@paf31 paf31 modified the milestones: 0.9.0, 0.10.0 Apr 10, 2016
@paf31 paf31 mentioned this pull request Apr 13, 2016
15 tasks
@paf31 paf31 modified the milestone: 0.9.0 Apr 20, 2016
@paf31
Copy link
Copy Markdown
Contributor Author

paf31 commented Apr 24, 2016

Rebased.

@garyb
Copy link
Copy Markdown
Member

garyb commented Apr 24, 2016

Shall we start merging these and call 0.8 done? Doesn't seem like there are any pressing issues that would necessitate a 0.8.6.

@garyb
Copy link
Copy Markdown
Member

garyb commented Apr 24, 2016

Oh, and 👍 😄

@paf31
Copy link
Copy Markdown
Contributor Author

paf31 commented Apr 24, 2016

I'd like to close 0.8.*, yeah. Maybe we can continue in a branch if necessary, or work in a branch for 0.9.

@garyb
Copy link
Copy Markdown
Member

garyb commented Apr 24, 2016

Sounds good to me. :shipit:

@hdgarrood
Copy link
Copy Markdown
Contributor

/cc @ethul just in case (I think the purescript webpack stuff uses --require-path)?

@ethul
Copy link
Copy Markdown

ethul commented Apr 25, 2016

Yes, noted. Thanks!

On Monday, 25 April 2016, Harry Garrood notifications@github.com wrote:

/cc @ethul https://github.com/ethul just in case (I think the
purescript webpack stuff uses --require-path)?


You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
#1996 (comment)

paf31 added a commit that referenced this pull request Apr 30, 2016
@paf31 paf31 closed this Apr 30, 2016
@paf31 paf31 deleted the no-require-path branch April 30, 2016 21:38
archaeron pushed a commit to archaeron/purescript that referenced this pull request Apr 6, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants