Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

remove Stack class -- unused #170

Closed
NickleDave opened this issue May 14, 2022 · 1 comment
Closed

remove Stack class -- unused #170

NickleDave opened this issue May 14, 2022 · 1 comment
Projects
ENH

Comments

@NickleDave
Copy link
Collaborator

already have #14 open as an issue but for now there's no code that actually uses the Stack class that's in its own module, so just remove it
@NickleDave NickleDave added this to To do in ENH May 14, 2022
@NickleDave NickleDave mentioned this issue May 14, 2022
Closed
@NickleDave
Copy link
Collaborator Author

And there is also #42 -- going to leave that one open as a possible enhancement but go ahead and remove for now, can always dig this class up from history if we want it

@NickleDave NickleDave mentioned this issue May 14, 2022
Closed
25 tasks
@NickleDave NickleDave moved this from To do to In progress in ENH May 15, 2022
@NickleDave NickleDave moved this from In progress to Done in ENH May 15, 2022
NickleDave added a commit that referenced this issue May 15, 2022
@NickleDave
Merge pull request #172 from vocalpy/remove-stack-class
cbf3329 
CLN: Remove crowsetta/stack.py, fix #170
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
ENH
Done
Milestone
No milestone
Development

No branches or pull requests
1 participant
@NickleDave