Conversation
msal4j-brokers/pom.xml
Outdated
| <developers> | ||
| <developer> | ||
| <id>msopentech</id> | ||
| <name>Microsoft Open Technologies, Inc.</name> |
There was a problem hiding this comment.
It was mentioned in a comment on your PR for the build script in the interop layer that 'Microsoft Open Technologies, Inc.' seems like a weird thing to put here, and honestly I'm not sure why this is even in our pom.xml for MSAL Java.
The tag is meant to be an optional documentation thing to just describe the developers, and I think we can just remove it from our pom's since it's not really necessary: https://maven.apache.org/pom.html#Developers
There was a problem hiding this comment.
Good point. It is interesting that some of the tags are optional but were giving me errors if not included. I dont remember if developer tag was one of them but to make it work, I just included everything from msal java pom xml. Lets talk and research more on why it was included in the first place before removing them. If they can be removed, we will remove them from both the modules.
This PR makes following changes -