Closed TrestanSimon closed 1 year ago
Thanks for opening this, I appreciate the time it takes to put something like this together. Because I don't have much time to work on and maintain pfsspy any more, I've decided to call it feature complete, and not add any new functionality. So unfortunately that means I'm going to close this without merging.
Thanks for opening this, I appreciate the time it takes to put something like this together. Because I don't have much time to work on and maintain pfsspy any more, I've decided to call it feature complete, and not add any new functionality. So unfortunately that means I'm going to close this without merging.
@dstansby Thanks for this message. I am aware that this project is and has been feature complete (from the docs) since before I opened the initial PR https://github.com/dstansby/pfsspy/pull/357 for this feature in August 2022 (cf with date of 66ac712b3f947bc5291cd7e6a182115c476b848c). I totally understand and respect this decision of course. However, you let me know that this feature was welcome via a very kind Google Groups message and https://github.com/dstansby/pfsspy/pull/357#pullrequestreview-1095129671 in September 2022, and also reaffirmed this two months ago in https://github.com/dstansby/pfsspy/pull/357#issuecomment-1508948460. So, I am kind of confused why you closed this PR now, and I would greatly appreciate it if you could let me know if it is no longer welcome (which, again, I totally understand and respect if so). :)
This is a continuation of PR357 previously closed due to inactivity. Apologies for this inactivity, I have been very busy with school-related projects and have not had time to return to complete this.
I have now addressed the comments in https://github.com/dstansby/pfsspy/pull/357#pullrequestreview-1095129671:
br
andbr_outer
(https://github.com/dstansby/pfsspy/pull/362/commits/3520afe9631b07fb52a4aded13e50317b9c7fb27)