search

ispras / clouni

Cloud Unifier Tool for Service Orchestration
Apache License 2.0
5 stars 8 forks source link

Sync tosca-parser with upstream #34

Closed abitrolly closed 2 years ago

abitrolly commented 2 years ago

Two projects are out of sync, but it doesn't looks like there are too much changes between them.

https://github.com/openstack/tosca-parser/compare/master...bura2017:master

Why not send PR upstream?

bura2017 commented 2 years ago

Pull requests were send to the upstream project tosca-parser, but it takes time for OpenStack developers to accept them

abitrolly commented 2 years ago

Looks like PRs are not here https://github.com/openstack/tosca-parser/pulls

Where are they located?

bura2017 commented 2 years ago

All contributions to the OpenStack projects must be made through https://review.opendev.org/q/status:open+-is:wip Pull requests to github are ignored

bura2017 commented 2 years ago

Actually all my contribution commits were already added to the upstream, so I checked and tosca-parser can be used from upstream for now

abitrolly commented 2 years ago

Looks like this https://review.opendev.org/c/openstack/tosca-parser/+/735624

But for some reason the GitHub diff is still showing changes.

bura2017 commented 2 years ago

Ok, I have updated our forked tosca-parser repo, it had some small changes in setup.cfg and some other non code files