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.

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

Adjusted documentation about packages to v1/v2 instead of old/new #3237

Draft
wants to merge 12 commits into
base: master
Choose a base branch
from

Conversation

craigcomstock
Copy link
Contributor

We felt in discussion internally that v1/v2 was easier to understand especially if we also include how they are implemented:

v1 packages promise (package_method)
v2 packages promise (package_module)

This leaves easier room for a v3 if that ever needs to happen.

v1(old) packages promise still has several advantages:

  • support for more different package managers such as pip, brew, etc.
  • support for bulk operations by default

Ticket: CFE-4315
Changelog: title

We felt in discussion internally that v1/v2 was easier to understand especially if we also include how they are implemented:

v1 packages promise (package_method)
v2 packages promise (package_module)

This leaves easier room for a v3 if that ever needs to happen.

v1(old) packages promise still has several advantages:
- support for more different package managers such as pip, brew, etc.
- support for bulk operations by default

Ticket: CFE-4315
Changelog: title
@craigcomstock
Copy link
Contributor Author

@cf-bottom jenkins please

@craigcomstock
Copy link
Contributor Author

this is related to cfengine/core#5434 (CFE-4315) but isn't dependent on it from a code point of view so am not putting "together" so I can get a fast docs build.

@craigcomstock
Copy link
Contributor Author

@cf-bottom jenkins another go please

@craigcomstock
Copy link
Contributor Author

@cf-bottom jenkins

@craigcomstock
Copy link
Contributor Author

@cf-bottom jenkins

@craigcomstock
Copy link
Contributor Author

Build Status

@craigcomstock craigcomstock marked this pull request as draft February 23, 2024 14:01
@craigcomstock
Copy link
Contributor Author

Build Status

@craigcomstock
Copy link
Contributor Author

Build Status

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants