ci: remove change type from uploaded payload size data (#33987)
The change type was only recorded for `aio/` and was not correct anyway. For example: - It considered `package.json` changes as `application` (even if only `package.json` and `yarn.lock` had changed). - It failed to account for changes in `@angular/*` dependencies, when using the locally built Angular packages (instead reporting them as `other`). - It only looked at the last commit, so it failed to provide accurate information for multi-commit builds (which are rare, but possible). For the above reasons (and because there is no straight-forward way of fixing it), this commit removes the change type from the uploaded data. If necessary, it is still possible to find the type of changes from the uploaded info (e.g. extract the associated commits and look at their changes using git). PR Close #33987
This commit is contained in:

committed by
Matias Niemelä

parent
efd626c4bc
commit
abb8c90df9
@ -64,7 +64,7 @@ for testDir in ${TEST_DIRS}; do
|
||||
yarn build
|
||||
fi
|
||||
|
||||
trackPayloadSize "$testDir" "dist/*.js" true false "${basedir}/integration/_payload-limits.json"
|
||||
trackPayloadSize "$testDir" "dist/*.js" true "${basedir}/integration/_payload-limits.json"
|
||||
fi
|
||||
|
||||
# remove the temporary node modules directory to keep the source folder clean.
|
||||
@ -73,5 +73,5 @@ for testDir in ${TEST_DIRS}; do
|
||||
done
|
||||
|
||||
if $CI; then
|
||||
trackPayloadSize "umd" "../dist/packages-dist/*/bundles/*.umd.min.js" false false
|
||||
trackPayloadSize "umd" "../dist/packages-dist/*/bundles/*.umd.min.js" false
|
||||
fi
|
||||
|
Reference in New Issue
Block a user