Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Modular structure #375

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

Draft
wants to merge 4 commits into
base: main
Choose a base branch
from
Draft

Modular structure #375

wants to merge 4 commits into from

Conversation

jeff5
Copy link
Member

@jeff5 jeff5 commented Mar 14, 2025

This PR, when it has all its change sets, will restructure the Jython 3 code into a Java module core. The objective is to control visibility of objects that will become API, while allowing us to use package structure internally to the project. (Without Java modularity, everything we make public for re-use between packages internally, becomes public externally.)

It is intended that there be more than one module eventually, but always a small number.

We are applying some off-repo learning (in VSJ4). The first step (adding module definitions) does not achieve the objective: public classes must be allocated to packages that are either exported or not, and in some cases a class must be divided between them. The off-repo work also advanced the functional state of the implementation so we shall bring that in too.

This gets quite difficult so the PR will be draft for a while.

jeff5 added 2 commits March 14, 2025 08:12
Script changes preparatory to introducing modules (in the layout we
found to work in VSJ4).
@jeff5 jeff5 marked this pull request as draft March 14, 2025 09:16
jeff5 added 2 commits March 14, 2025 18:17
This allows us to test creation of the several kinds of Javadoc
publication.
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.

1 participant