Skip to content

rules/sdk: sdk allow unsafe+*/rand in specific packages#46

Merged
odeke-em merged 1 commit intomasterfrom
rules-sdk-allow-unsafe+rand-in-specific-packages
Sep 24, 2022
Merged

rules/sdk: sdk allow unsafe+*/rand in specific packages#46
odeke-em merged 1 commit intomasterfrom
rules-sdk-allow-unsafe+rand-in-specific-packages

Conversation

@odeke-em
Copy link
Contributor

There are some package whose core functionality relies on unsafe imports as well randomization code for example:

  • codegen
  • crypto/*
  • simapp
  • simulation
  • testutil and other testing code

thus allow them to fly with unsafe imports.

Fixes #44

@odeke-em
Copy link
Contributor Author

Just a kind FYI for @ebuchman @kirbyquerby @elias-orijtech @willpoint

There are some package whose core functionality relies on
unsafe imports as well randomization code for example:
* codegen
* crypto/*
* simapp
* simulation
* testutil and other testing code

thus allow them to fly with unsafe imports.

Fixes #44
@odeke-em odeke-em force-pushed the rules-sdk-allow-unsafe+rand-in-specific-packages branch from 1901d6b to 7741996 Compare September 24, 2022 02:10
@odeke-em odeke-em merged commit 7741996 into master Sep 24, 2022
@odeke-em odeke-em deleted the rules-sdk-allow-unsafe+rand-in-specific-packages branch September 24, 2022 02:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

rules/sdk: ignore unsafe imports inside any simapp or simulation, cryptographic, testing related code

1 participant