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

Update openapi-extractor to latest #11314

Merged
merged 1 commit into from
Nov 27, 2023

Conversation

nickvergessen
Copy link
Member

☑️ Resolves

  • Fix failing build:
$ make openapi-spec 
cd build && sh get-server-sources.sh 


Fetching source for master

remote: Enumerating objects: 11551, done.
remote: Counting objects: 100% (8964/8964), done.
remote: Compressing objects: 100% (3345/3345), done.
remote: Total 4544 (delta 2592), reused 1695 (delta 1103), pack-reused 0
Empfange Objekte: 100% (4544/4544), 12.92 MiB | 4.05 MiB/s, fertig.
Löse Unterschiede auf: 100% (2592/2592), abgeschlossen mit 1794 lokalen Objekten.
Von https://github.com/nextcloud/server
 + 5821f884e8...b3e24a6bf2 master     -> origin/master  (Aktualisierung erzwungen)
 + e867ca0c2e...b4d6db5a88 stable24   -> origin/stable24  (Aktualisierung erzwungen)
Bereits auf 'master'
Ihr Branch und 'origin/master' sind divergiert,
und haben jeweils 1 und 1 unterschiedliche Commits.
  (verwenden Sie "git pull", wenn Sie den Remote-Branch in Ihren integrieren wollen)
HEAD ist jetzt bei b3e24a6bf2 Merge pull request #41764 from nextcloud/nickvergessen-patch-1
git submodule update --init
cd build/openapi-extractor && composer install
Installing dependencies from lock file (including require-dev)
Verifying lock file contents can be installed on current platform.
Nothing to install, update or remove
Generating autoload files
1 package you are using is looking for funding.
Use the `composer fund` command to find out more!
cd build && ./openapi-extractor/merge-specs \
	--core server/core/openapi.json \
	--merged ../developer_manual/_static/openapi.json \
	$(ls server/apps/*/openapi.json)
PHP Warning:  Undefined array key "properties" in /home/nickv/Nextcloud/Repositories/documentation/build/openapi-extractor/merge-specs on line 106
PHP Fatal error:  Uncaught TypeError: array_merge_recursive(): Argument #2 must be of type array, null given in /home/nickv/Nextcloud/Repositories/documentation/build/openapi-extractor/merge-specs:106
Stack trace:
#0 /home/nickv/Nextcloud/Repositories/documentation/build/openapi-extractor/merge-specs(106): array_merge_recursive()
#1 /home/nickv/Nextcloud/Repositories/documentation/build/openapi-extractor/merge-specs(67): collectCapabilities()
#2 {main}
  thrown in /home/nickv/Nextcloud/Repositories/documentation/build/openapi-extractor/merge-specs on line 106
make: *** [Makefile:35: openapi-spec] Fehler 255

@nickvergessen
Copy link
Member Author

/backport to stable28

@nickvergessen
Copy link
Member Author

/backport to stable27

@nickvergessen nickvergessen merged commit 0993945 into master Nov 27, 2023
8 checks passed
@nickvergessen nickvergessen deleted the bugfix/noid/update-openapi-extractor branch November 27, 2023 13:11
@provokateurin
Copy link
Member

It should instead use the one that is in the composer.lock of server to always have the correct one.

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

Successfully merging this pull request may close these issues.

3 participants