pesde-pkg/pesde
1
1
Fork 0
mirror of https://github.com/pesde-pkg/pesde.git synced 2025-04-05 11:20:55 +01:00
Code Issues Projects Releases Packages Wiki Activity

feat: remove cli side dependency checks

Browse source
This commit is contained in:
daimond113 2025-01-09 22:36:53 +01:00
parent e61aeb5da0
commit 217ca238ff
No known key found for this signature in database
GPG key ID: 3A8ECE51328B513C
2 changed files with 2 additions and 20 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
CHANGELOG.md
View file

@ -13,6 +13,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
- Add path dependencies by @daimond113 - Add path dependencies by @daimond113
- Inherit pesde-managed scripts from workspace root by @daimond113 - Inherit pesde-managed scripts from workspace root by @daimond113
- Allow using binaries from workspace root in member packages by @daimond113 - Allow using binaries from workspace root in member packages by @daimond113
- Add yanking & deprecating by @daimond113
### Fixed ### Fixed
- Install dev packages in prod mode and remove them after use to allow them to be used in scripts by @daimond113 - Install dev packages in prod mode and remove them after use to allow them to be used in scripts by @daimond113
@ -25,6 +26,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
### Removed ### Removed
- Remove old includes format compatibility by @daimond113 - Remove old includes format compatibility by @daimond113
- Remove data redundacy for workspace package references by @daimond113 - Remove data redundacy for workspace package references by @daimond113
- Remove dependency checks from CLI in publish command in favor of registry checks by @daimond113
### Performance ### Performance
- Use `Arc` for more efficient cloning of multiple structs by @daimond113 - Use `Arc` for more efficient cloning of multiple structs by @daimond113

20
src/cli/commands/publish.rs
View file

@ -9,7 +9,6 @@ use pesde::{
matching_globs, matching_globs,
scripts::ScriptName, scripts::ScriptName,
source::{ source::{
git_index::GitBasedSource,
pesde::{specifier::PesdeDependencySpecifier, PesdePackageSource}, pesde::{specifier::PesdeDependencySpecifier, PesdePackageSource},
specifiers::DependencySpecifiers, specifiers::DependencySpecifiers,
traits::{GetTargetOptions, PackageRef, PackageSource, RefreshOptions, ResolveOptions}, traits::{GetTargetOptions, PackageRef, PackageSource, RefreshOptions, ResolveOptions},
@ -666,25 +665,6 @@ info: otherwise, the file was deemed unnecessary, if you don't understand why, p
); );
} }
let deps = manifest.all_dependencies().context("dependency conflict")?;
if let Some((disallowed, _)) = deps.iter().find(|(_, (spec, _))| match spec {
DependencySpecifiers::Pesde(spec) => {
!config.other_registries_allowed.is_allowed_or_same(
source.repo_url().clone(),
gix::Url::try_from(spec.index.as_deref().unwrap()).unwrap(),
)
}
DependencySpecifiers::Git(spec) => !config.git_allowed.is_allowed(spec.repo.clone()),
#[cfg(feature = "wally-compat")]
DependencySpecifiers::Wally(spec) => !config
.wally_allowed
.is_allowed(gix::Url::try_from(spec.index.as_deref().unwrap()).unwrap()),
_ => false,
}) {
anyhow::bail!("dependency `{disallowed}` is not allowed on this index");
}
if self.dry_run { if self.dry_run {
fs::write("package.tar.gz", archive).await?; fs::write("package.tar.gz", archive).await?;