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

odpi/egeria#6907 master->main rename #778

Merged
merged 5 commits into from Oct 20, 2022
Merged

Conversation

planetf1
Copy link
Member

Signed-off-by: Nigel Jones nigel.l.jones+git@gmail.com

Updating master->main

Signed-off-by: Nigel Jones <nigel.l.jones+git@gmail.com>
@sonatype-lift
Copy link

sonatype-lift bot commented Oct 20, 2022

⚠️ 21 God Classes were detected by Lift in this project. Visit the Lift web console for more details.

@@ -1596,7 +1596,7 @@
DynamicPropertyReader reader = new DynamicPropertyReader(getPOJOForType(type), property);
typeAndPropertyToAccessor.put(key, reader);
} catch (IllegalArgumentException e) {
log.warn("Unable to setup an accessor for property '{}' on type '{}' - this property will be entirely ignored. If this is a custom property, see https://github.com/odpi/egeria-connector-ibm-information-server/tree/master/igc-clientlibrary#using-your-own-asset-types for how to add your own properties.", property, type, e);
log.warn("Unable to setup an accessor for property '{}' on type '{}' - this property will be entirely ignored. If this is a custom property, see https://github.com/odpi/egeria-connector-ibm-information-server/tree/main/igc-clientlibrary#using-your-own-asset-types for how to add your own properties.", property, type, e);

Check failure

Code scanning / CodeQL

Insertion of sensitive information into log files

This [potentially sensitive information](1) is written to a log file. This [potentially sensitive information](2) is written to a log file.
Signed-off-by: Nigel Jones <nigel.l.jones+git@gmail.com>
Signed-off-by: Nigel Jones <nigel.l.jones+git@gmail.com>
Signed-off-by: Nigel Jones <nigel.l.jones+git@gmail.com>
@planetf1
Copy link
Member Author

@alexandra-bucur @lpalashevski This PR is to update some code refs for 'master' to use 'main' instead.

However I note that all PR builds are failing currently. I initially saw a memory issue, so I changed the mem available for tests. However I continue to get a test failure

Is this something you are aware of?

@alexandra-bucur
Copy link
Contributor

Hi @planetf1! The failure of the tests is related to some changes that were added in Egeria's models. I added the new fields in the mappers, but they are inside #777. Can this PR wait for a review on the other one?

@lpalashevski
Copy link
Contributor

Yes. @planetf1 please put the rename for this repo on hold until we fix the build with #777. Thanks!

@planetf1
Copy link
Member Author

@lpalashevski The rename is already done for the repository.

This PR is just to update files within the repository & build script. We'll need to get just the build updated so we can push through the cicd changes - or you can cherry-pick the commits here into your proposal

Signed-off-by: Alexandra Bucur <alexandra-nona.bucur@ing.com>
@planetf1 planetf1 merged commit 3bd3577 into odpi:main Oct 20, 2022
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.

None yet

3 participants