-
-
Notifications
You must be signed in to change notification settings - Fork 933
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
SAK-50556 Global update dependencies for Sakai 25 #12924
Conversation
LTI stuff LGTM |
@@ -31,9 +31,9 @@ public void testInstantParse() throws Exception { | |||
i = InstantUtil.parseGMTFormats(txt); | |||
assertEquals(i.toString(), "2008-04-09T23:55:38Z"); | |||
|
|||
txt = "Fri Feb 15 14:45:01 2013"; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this is a very strange date @csev .... where did you find it?
master/pom.xml
Outdated
<dependency> | ||
<groupId>commons-collections</groupId> | ||
<artifactId>commons-collections</artifactId> | ||
<version>3.2.2</version> | ||
<scope>provided</scope> | ||
</dependency> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
see SAK-49906, this is transitively needed
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
needed by who?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
i think i understand.... you'd rather just declare it centrally instead of having old tool dependencies like velocity and tomahawk bring it in transitively
No description provided.