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

Patch to obtain changes made in #166: attrs condition check in StackFrame #390

Merged
merged 20 commits into from
Aug 11, 2023

Conversation

varad64
Copy link
Contributor

@varad64 varad64 commented Aug 11, 2023

Updates the condition in StackFrame class and StackFrameTest tests the change. Here is a link to the original PR containing the changes.

varad64 and others added 20 commits June 23, 2023 01:27
…part of javapathfinder#274

- Fixes three failing tests from DateFormatTest and the sole failing test from SimpleDateFormatTest class
 - Build Scan plugin has been replaced with the Gradle Enterprise plugin to ensure compatibility with gradle v6.0

 - Deprecated plugin conventions have been replaced with extensions to ensure compatibility with gradle v9.0

 - Tasks now follow the Task Configuration Avoidance API to ensure forward compatibility

 - Deprecated destinationDir of AbstractCompile replaced with destinationDirectory to ensure forward compatibility

 - Deprecated outputDir of SourceDirectorySet replaced with destinationDirectory to ensure forward compatibility

 - Explicit dependencies declared on the copyResources task to avoid dependency management related errors
…e following Strong Encapsulation:

 - add-opens is used to access the packages java.lang and java.util present in the java.base module
@varad64 varad64 changed the base branch from java-10-gradle to master August 11, 2023 14:22
@varad64 varad64 changed the base branch from master to java-10-gradle August 11, 2023 14:23
@cyrille-artho cyrille-artho merged commit 4e421c9 into javapathfinder:java-10-gradle Aug 11, 2023
2 checks passed
@varad64
Copy link
Contributor Author

varad64 commented Aug 11, 2023

Strangely, the commits that have already been merged still show up. Apologies for the messy commit history.

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.

2 participants