Skip to content

Make read-only predicate support Resource and EAnnotation#1358

Merged
AxelRICHARD merged 1 commit intoeclipse-syson:mainfrom
flatombe:dev/FLA/1357
Jun 12, 2025
Merged

Make read-only predicate support Resource and EAnnotation#1358
AxelRICHARD merged 1 commit intoeclipse-syson:mainfrom
flatombe:dev/FLA/1357

Conversation

@flatombe
Copy link
Copy Markdown
Contributor

PLEASE READ ALL ITEMS AND CHECK ONLY RELEVANT CHECKBOXES BELOW

Project management

  • Has the pull request been added to the relevant milestone?
  • Have the priority: and pr: labels been added to the pull request? (In case of doubt, start with the labels priority: low and pr: to review later)
  • Have the relevant issues been added to the pull request?
  • Have the relevant labels been added to the issues? (area:, type:)
  • Have the relevant issues been added to the same project milestone as the pull request?

Changelog and release notes

  • Has the CHANGELOG.adoc + doc/content/modules/user-manual/pages/release-notes/YYYY.MM.0.adoc been updated to reference the relevant issues?
  • Have the relevant API breaks been described in the CHANGELOG.adoc + doc/content/modules/user-manual/pages/release-notes/YYYY.MM.0.adoc?
  • In case of a change with a visual impact, are there any screenshots in the doc/content/modules/user-manual/pages/release-notes/YYYY.MM.0.adoc?
  • In case of a key change, has the change been added to Key highlights section in doc/content/modules/user-manual/pages/release-notes/YYYY.MM.0.adoc?
  • Are the new / upgraded dependencies mentioned in the relevant section of the CHANGELOG.adoc + doc/content/modules/user-manual/pages/release-notes/YYYY.MM.0.adoc?

@AxelRICHARD
Copy link
Copy Markdown
Member

#1339 has already been merged. Is it normal that you have a commit in this PR that references this issue?

@flatombe
Copy link
Copy Markdown
Contributor Author

My bad, I was not up to date on my branch :-)
This should be fixed now.

@AxelRICHARD
Copy link
Copy Markdown
Member

Tests are missing. Could you please add some?

@flatombe flatombe force-pushed the dev/FLA/1357 branch 2 times, most recently from ebdd13f to 43d0db6 Compare June 11, 2025 15:19
@flatombe
Copy link
Copy Markdown
Contributor Author

I've added the unit tests for the whole class, and updated the release notes so the build should succeed now.

* SysONReadOnlyObjectPredicateDelegate now supports Resource and
EAnnotation.
* Add unit tests for SysONReadOnlyObjectPredicateDelegate.
* Update changelog and release notes accordingly.

Bug: eclipse-syson#1357
Signed-off-by: Florent Latombe <florent.latombe@obeo.fr>
@AxelRICHARD AxelRICHARD merged commit 5047906 into eclipse-syson:main Jun 12, 2025
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

SysONReadOnlyObjectPredicateDelegate does not handle Resources or EAnnotations

2 participants