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

[FER] replace whitelist/blacklist with allowlist/blocklist #37

Closed
rexut opened this issue Jul 3, 2021 · 1 comment · Fixed by #40 or #42
Closed

[FER] replace whitelist/blacklist with allowlist/blocklist #37

rexut opened this issue Jul 3, 2021 · 1 comment · Fixed by #40 or #42
Assignees
@tobiaskaestner @rexut
Labels
enhancement New feature or request
Milestone
v2.5.1

Comments

@rexut
Copy link
Member

rexut commented Jul 3, 2021 •
edited
Loading

Is your enhancement proposal related to a problem? Please describe.
Per zephyrproject-rtos/west#440 and in common https://twitter.com/dhh/status/1032050325513940992, I'd like for Bridle to use the correct keywords in west.yml file.

Describe the solution you'd like
West 0.9 is the minimum version, we can use 'allowlist' or 'blocklist'. Set manifest requires west 0.10 or later to cache potential fixes, use the new keywords, the submodules feature, also be prepared for the new 'filter' keyword..

Describe alternatives you've considered
Non.

Additional context
Non.
@rexut rexut added the enhancement New feature or request label Jul 3, 2021
@rexut
Copy link
Member Author

rexut commented Jul 3, 2021

When we upgrade the minimum version for west, I'll go also over all Python requirements and adjust freeze versions and the documentation on demand.

rexut added a commit that referenced this issue Jul 3, 2021
@rexut
manifest: switch to 'allowlist'
27dff88 
Now that west 0.9 is the minimum version, we can use
'allowlist' when specifying the projects to import
from the Zephyr repository.

issues: #37
Signed-off-by: Stephan Linz <linz@li-pro.net>
rexut added a commit that referenced this issue Jul 3, 2021
@rexut
requirements: bump differet PyPi packages
d34a3a0 
- west to 0.10.0, or later
- ecdsa to 0.17.0, or later
- pylint to 2.9.0, or later
- regex to 2021.7.1, or later
- pygments to 2.9, or later

- commonmark to 0.9.1
- sphinxcontrib-mscgen to 0.6
- sphinx-tabs to 3.1.0

issues: #37
Signed-off-by: Stephan Linz <linz@li-pro.net>
@rexut rexut linked a pull request Jul 3, 2021 that will close this issue
preparations for upgrade to Zephyr v2.6 #40
Merged
@rexut rexut modified the milestones: V2.5.1, v2.6.0, v2.5.0 Jul 3, 2021
@rexut rexut closed this as completed in #40 Jul 3, 2021
rexut added a commit that referenced this issue Jul 3, 2021
@rexut
manifest: switch to 'allowlist'
dfdc4c6 
Now that west 0.9 is the minimum version, we can use
'allowlist' when specifying the projects to import
from the Zephyr repository.

issues: #37
Signed-off-by: Stephan Linz <linz@li-pro.net>
rexut added a commit that referenced this issue Jul 3, 2021
@rexut
requirements: bump differet PyPi packages
c4a9aba 
- west to 0.10.0, or later
- ecdsa to 0.17.0, or later
- pylint to 2.9.0, or later
- regex to 2021.7.1, or later
- pygments to 2.9, or later

- commonmark to 0.9.1
- sphinxcontrib-mscgen to 0.6
- sphinx-tabs to 3.1.0

issues: #37
Signed-off-by: Stephan Linz <linz@li-pro.net>
rexut added a commit that referenced this issue Jul 3, 2021
@rexut
manifest: switch to 'allowlist'
142944f 
Now that west 0.9 is the minimum version, we can use
'allowlist' when specifying the projects to import
from the Zephyr repository.

issues: #37
Signed-off-by: Stephan Linz <linz@li-pro.net>
rexut added a commit that referenced this issue Jul 3, 2021
@rexut
requirements: bump differet PyPi packages
18ca96a 
- west to 0.10.0, or later
- ecdsa to 0.17.0, or later
- pylint to 2.9.0, or later
- regex to 2021.7.1, or later
- pygments to 2.9, or later

- commonmark to 0.9.1
- sphinxcontrib-mscgen to 0.6
- sphinx-tabs to 3.1.0

issues: #37
Signed-off-by: Stephan Linz <linz@li-pro.net>
@rexut rexut linked a pull request Jul 3, 2021 that will close this issue
pick latest upgrade preparations from main #42
Merged
tobiaskaestner pushed a commit that referenced this issue Jul 9, 2021
@rexut @tobiaskaestner
manifest: switch to 'allowlist'
b426593 
Now that west 0.9 is the minimum version, we can use
'allowlist' when specifying the projects to import
from the Zephyr repository.

issues: #37
Signed-off-by: Stephan Linz <linz@li-pro.net>
tobiaskaestner pushed a commit that referenced this issue Jul 9, 2021
@rexut @tobiaskaestner
requirements: bump differet PyPi packages
2f7ae97 
- west to 0.10.0, or later
- ecdsa to 0.17.0, or later
- pylint to 2.9.0, or later
- regex to 2021.7.1, or later
- pygments to 2.9, or later

- commonmark to 0.9.1
- sphinxcontrib-mscgen to 0.6
- sphinx-tabs to 3.1.0

issues: #37
Signed-off-by: Stephan Linz <linz@li-pro.net>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@tobiaskaestner tobiaskaestner

@rexut rexut

Labels
enhancement New feature or request
Projects
None yet
Milestone
v2.5.1
2 participants
@tobiaskaestner @rexut