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

Better world: First refactor of the lstags #63

Closed
4 tasks done
ivanilves opened this issue Sep 26, 2017 · 7 comments
Closed
4 tasks done

Better world: First refactor of the lstags #63

ivanilves opened this issue Sep 26, 2017 · 7 comments
Assignees
@vonrabbe @ivanilves
Labels
critical enhancement

Comments

@ivanilves
Copy link
Owner

ivanilves commented Sep 26, 2017 •
edited
Loading

We have reached the state when we need to:

  • stop adding new functionality for a while.
  • work more on automated testing, code quality and various fixes.
  • add performance, benefit more of concurrency Golang gives us [almost] for free.

TODO

  • Add black-box shell tests for "image pull" functionality.
  • Fix lame ("double"?) help messages.
  • Make registry and operations event more concurrent.
  • Fix authenticated pull.

GIF
@ivanilves
Copy link
Owner Author

#65

  • Make registry and operations event more concurrent.

@ivanilves
Copy link
Owner Author

#66

  • Fix lame ("double"?) help messages.

@ivanilves
Copy link
Owner Author

💡 We need to test pull of some scratch-based repo inside our integration tests!

ivanilves added a commit that referenced this issue Oct 3, 2017
@ivanilves
Merge pull request #66 from /issues/63/cli-help
0cc7eb0 
Fix fuzzy CLI help
@ivanilves
Copy link
Owner Author

#70

  • Fix authenticated pull

@ivanilves
Copy link
Owner Author

I would better get back to milestones for this...

@ivanilves ivanilves mentioned this issue Oct 8, 2017
Merged
@ivanilves
Copy link
Owner Author

Here we work to finish the issue: #71

ivanilves added a commit that referenced this issue Oct 9, 2017
@ivanilves
Merge pull request #71 from /issues/63/refactor
372b19e 
First refactor of L/S tags
@ivanilves
Copy link
Owner Author

I consider this done. Scope is reduced, but it is done.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@vonrabbe vonrabbe

@ivanilves ivanilves

Labels
critical enhancement
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@vonrabbe @ivanilves