Skip to content
This repository was archived by the owner on Nov 7, 2026. It is now read-only.

Create a basic project structure - #4

Open
uchitsa wants to merge 10 commits into
objectionary:masterfrom
uchitsa:2
Open

uchitsa wants to merge 10 commits into
objectionary:masterfrom
uchitsa:2

Conversation

@uchitsa

@uchitsa uchitsa commented Sep 12, 2025

Copy link
Copy Markdown
Member

No description provided.

@uchitsa

uchitsa commented Sep 17, 2025

Copy link
Copy Markdown
Member Author

Issue #2

@uchitsa

uchitsa commented Sep 17, 2025

Copy link
Copy Markdown
Member Author

@yegor256 hello, please take a look

@yegor256 yegor256 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The structure does not yet match issue #2, and the build cannot work as configured. Please fix the points below, then run mvn clean compile locally and paste its result here, since the repository has no CI to check it.

@@ -0,0 +1,2 @@
# This is the main time object that provides access to all time-related functionality
[] > time

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The directory is named org.eolang, a single folder with a dot in its name. Issue #2 asks for src/main/eo/org/eolang/time.eo, two nested folders. Please move the file there.

@@ -0,0 +1,2 @@
# This is the main time object that provides access to all time-related functionality

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The file has no +package org.eolang meta line, so the object will not land in the org.eolang package. Please add +package org.eolang (and the +home and +version metas used in other objectionary repositories) above the comment, separated by an empty line.

Comment thread pom.xml Outdated

<properties>
<project.build.sourceEncoding>UTF-8</project.build.sourceEncoding>
<eo.version>0.29.0</eo.version>

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

eo-runtime is pinned to 0.29.0, while eo-maven-plugin is 0.58.3. The runtime and the plugin must share one version, otherwise the transpiled Java code will not compile against the runtime. Please use ${eo.version} for the plugin too and set it to 0.58.3 (or the latest release).

Comment thread pom.xml Outdated
</plugin>
</plugins>
</build>
</project> No newline at end of file

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The file has no newline at the end. Please add one.

@yegor256

yegor256 commented Nov 6, 2026

Copy link
Copy Markdown
Member

@uchitsa, dependency assessment is tracked in #8, assigned to @yegor256 for the Java baseline decision. Please keep #4 and draft #5 aligned: #4 now uses one EO 0.58.3 property, while #5 still mixes runtime 0.29.0 and plugin 0.58.3. After #4 lands, please carry that shared version into #5 when rebasing. EO 0.63.2 raises runtime bytecode to Java 17, so the current Java 11 target needs a decision before that paired upgrade. Structure and CI remain with their existing work; no competing scaffold or dependency PR has been opened.

This was referenced Nov 6, 2026
@yegor256

yegor256 commented Nov 6, 2026

Copy link
Copy Markdown
Member

@uchitsa, focused workflow CI is in #9 (draft until #4 provides the Maven project). It proposes mvn --batch-mode --no-transfer-progress clean verify, including your clean compile acceptance check plus test/package verification, using build JDK 17 and Maven 3.9.11 for EO 0.58.3. Please confirm that command for #4 and draft #5, or identify any required profile. This does not change the Java 11 compilation target or upgrade dependencies. Structure and implementation stay in your existing PRs; #5 remains draft. The CI prerequisite is assigned to @yegor256 in #9.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants