add public api markers

Review Request #3442 — Created Feb. 9, 2016 and submitted

molsen
pants
2710, 2908
pants-reviews
benjyw, jsirois, patricklaw, stuhood, zundel

add public api markers to several modules

  • goals
  • help
  • round_engine
  • fs
  • invalidation

CI green: https://travis-ci.org/pantsbuild/pants/builds/108170980

  • 23
  • 0
  • 0
  • 0
  • 23
Description From Last Updated
It doesn't seem like this or any of its methods should be public. This is an implementation detail of the ... ST stuhood
I don't think the constructor should be public here... tasks are provided a fully constructed RoundManager. ST stuhood
This class seems like a run_tracker.py implementation detail, so nothing in it should be public. ST stuhood
ditto runtracker implementation detail. ST stuhood
The constructor for Context shouldn't be public... tasks receive a fully constructed Context (and in tests, they should use the ... ST stuhood
Can't imagine a case where a plugin dev would use this. ST stuhood
This is required for use during task registration, IIRC. ST stuhood
Frequently used during task registration. ST stuhood
Is created by the engine. ST stuhood
Should be called by the engine. ST stuhood
ditto called by engine. ST stuhood
IIRC, this and most of the other methods on here should be accessed via context.log, so not many (none?) of ... ST stuhood
Constructor should not be public. ST stuhood
I'm not sure anything in here should be public... the class is consumed by tasks. ST stuhood
Ditto class internal only. ST stuhood
Ditto class internal only. ST stuhood
This class is internal only. ST stuhood
Everything in this class is private. ST stuhood
A user should not directly construct a VTS ST stuhood
...or a VT ST stuhood
This method is private. ST stuhood
This whole class is private. ST stuhood
This whole class is private, AFAIK. ST stuhood
MO
MO
PA
  1. Ship It!
  2. 
      
WI
  1. Ship It!
  2. 
      
MO
Review request changed

Status: Closed (submitted)

Change Summary:

commit f262a18a80164c7007cce6d2478d4c099a2e41d1

ST
  1. Sorry for the late review.

    Let's be much, much more conservative about these in the future: it's easier to expose something later than it is to unexpose it.

  2. src/python/pants/engine/round_engine.py (Diff revision 1)
     
     
     
     

    It doesn't seem like this or any of its methods should be public. This is an implementation detail of the engine.

  3. src/python/pants/engine/round_manager.py (Diff revision 1)
     
     
     
     

    I don't think the constructor should be public here... tasks are provided a fully constructed RoundManager.

  4. src/python/pants/engine/round_manager.py (Diff revision 1)
     
     
     

    I don't think this is supposed to be public (or, I've never seen it used by a task).

  5. This class seems like a run_tracker.py implementation detail, so nothing in it should be public.

  6. ditto runtracker implementation detail.

  7. src/python/pants/goal/context.py (Diff revision 1)
     
     
     
     

    The constructor for Context shouldn't be public... tasks receive a fully constructed Context (and in tests, they should use the test helper methods to get one).

  8. src/python/pants/goal/context.py (Diff revision 1)
     
     
     

    unsure if public?

  9. src/python/pants/goal/context.py (Diff revision 1)
     
     
     

    Unsure if public?

  10. src/python/pants/goal/context.py (Diff revision 1)
     
     

    Probably not public.

  11. src/python/pants/goal/context.py (Diff revision 1)
     
     
     
     
     
     

    Heh... it's whacky that this exists.

  12. src/python/pants/goal/error.py (Diff revision 1)
     
     
     

    Can't imagine a case where a plugin dev would use this.

  13. src/python/pants/goal/goal.py (Diff revision 1)
     
     

    This is required for use during task registration, IIRC.

  14. src/python/pants/goal/goal.py (Diff revision 1)
     
     
     
     
     
     
     
     
     

    Frequently used during task registration.

  15. src/python/pants/goal/products.py (Diff revision 1)
     
     
     
     

    IIRC, this should not be created by a user.

  16. src/python/pants/goal/run_tracker.py (Diff revision 1)
     
     
     
     

    Is created by the engine.

  17. src/python/pants/goal/run_tracker.py (Diff revision 1)
     
     
     

    Should be called by the engine.

  18. src/python/pants/goal/run_tracker.py (Diff revision 1)
     
     

    ditto called by engine.

  19. src/python/pants/goal/run_tracker.py (Diff revision 1)
     
     
     
     
     
     

    IIRC, this and most of the other methods on here should be accessed via context.log, so not many (none?) of them should be public.

  20. src/python/pants/goal/task_registrar.py (Diff revision 1)
     
     

    Constructor should not be public.

  21. I'm not sure anything in here should be public... the class is consumed by tasks.

  22. src/python/pants/help/help_formatter.py (Diff revision 1)
     
     
     
     
     
     
     
     
     

    Ditto class internal only.

  23. src/python/pants/help/help_info_extracter.py (Diff revision 1)
     
     
     
     

    Ditto class internal only.

  24. src/python/pants/help/help_printer.py (Diff revision 1)
     
     
     
     
     
     
     
     
     

    Ditto class internal only.

  25. src/python/pants/invalidation/build_invalidator.py (Diff revision 1)
     
     
     
     
     

    This class is internal only.

  26. Everything in this class is private.

  27. src/python/pants/invalidation/cache_manager.py (Diff revision 1)
     
     
     
     

    A user should not directly construct a VTS

  28. src/python/pants/invalidation/cache_manager.py (Diff revision 1)
     
     
     
     

    ...or a VT

  29. This method is private.

  30. This whole class is private.

  31. src/python/pants/invalidation/cache_manager.py (Diff revision 1)
     
     
     
     
     
     
     

    This whole class is private, AFAIK.

  32. 
      
Loading...