View source for GitHub Pull Requests


{{hint|If you want a pull request to be acted on, do not rely on the GitHub pull request tracker.<br>  For anything that is more than a couple of lines, you will almost certainly need to talk with us on [https://lists.sourceforge.net/lists/listinfo/audacity-devel audacity-devel mailing list].}}

==Labels==

We have these possible labels:

* '''nice idea''' - We like the idea. This is neither positive nor negative about the implementation of the idea.  We might not have reviewed or tried out the implementation yet.
* '''who can test?''' - This PR is not being progressed for want of someone to try it out, e.g different platform to most developers, or it is using non standard hardware.  Talk with us!
* '''do we want this?''' - Do we like the sound of this feature as described?  If our answer is yes, request becomes 'nice idea'.  If no, thanks but no thanks.
* '''lib-src or nyquist''' - This PR is about 'issues' with upstream code.  Closing these pull requests generally requires communication upstream.  Nyquist is easiest as upstream is in-house.  Other ones likely will take longer.  It's why we don't fix warnings on lib-src code.

==Milestones==
{{ednote|'''Gale 20Sep16:''' Should the link not be [https://github.com/audacity/audacity/milestone/1 1st January 2020 FAKE MILESTONE]? }}
We have one milestone:

{{hint|1=
[http://wiki.audacityteam.org/w/index.php?title=Github_Pull_Requests 1st January 2020 FAKE MILESTONE]

This isn't a milestone.  It's a way to tell people making any decent sized pull requests to please come over talk with us at the audacity-devel email list.  If you just rely on the GitHub pull request messages, you may find we ignore or close the pull request for what does not seem to you to be a good reason.  Please come and talk.

There is a bit more  on our wiki about how we use these pull requests.
}}


==Tables==

* Don't pay too much attention to the tables below.  

===On the way to being closed===
Open requests that we are probably going to close soon, or at least we hope so.  That could be close REJECT or close ACCEPT.

{| class="wikitable sortable" style="width: auto;background-color: #d8d8d8;"
|-
! Pull
! Link
! By
! Comments
|-
| 49
| [https://github.com/audacity/audacity/pull/49 Run chain from command line]
| Cory Cook
| Possibly an OK idea, but better done through using some of this code in scripting?
|-
| xx
| [https://github.com/audacity/audacity/commit/41de5b385b2ca721a71871fcdb00ec3fa4441b5b Add consts in FFmpeg]
| Max Kellermann
| row 2, cell 3
|-
| 75
| [https://github.com/audacity/audacity/pull/75 Use noinst_SCRIPTS for toplevel audacity script]
| 0-wiz-0
| James hasn't much of a clue what this is about.  It's short, and it looks like it should be a quick decision for someone who does know.
|}


===Closed but 'Nice Idea'===

Closed pull requests that we want to look at again later.  Possibly closed because either it or we are not ready for that pull request.

{| class="wikitable sortable" style="width: auto;background-color: #d8d8d8;"
|-
! Pull
! Link
! By
! Comments
|-
| 163
| [https://github.com/audacity/audacity/pull/163 OpenMP TrackArtist::DrawSpectrum] 3 times faster.
| Darrell Walisser
| Paul reviewed and was bothered mainly by changes which impact his as yet unmerged FishEye code.  Hoping/expecting a resubmit.
|-
| 135
| [https://github.com/audacity/audacity/pull/135 Create a hidden configuration option to disable the save prompt on exit #135]
| Bracketcc
| The actual pull request is enormous, so something has gone wrong somewhere, and it wasn't pulled.  Maybe look at #134 for the basic idea.
|-
| 142
| [https://github.com/audacity/audacity/pull/142 Parameter loading for batch EQ]
| Wave Motion
| Like the feature, but implementation not completed.
|}

[[Category:For Developers]]