Skip to content

fix(ENGKNOW-2760): norsql add project id support like we have for db links #74

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.

Sign up for GitHub

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

Jump to bottom
Merged
gmagnu merged 6 commits into from
Oct 24, 2025
Merged

fix(ENGKNOW-2760): norsql add project id support like we have for db links #74

gmagnu merged 6 commits into from
Oct 24, 2025

Conversation

@gmagnu
Copy link
Contributor

@gmagnu gmagnu commented Oct 24, 2025 •
edited
Loading

  • Move Constants class to SM, as it is only used there.
  • Move SQL specific constants and code back to SqlReplacer.
  • Add feature of global replace (default turned off)

@github-actions
Copy link

github-actions bot commented Oct 24, 2025 •
edited
Loading

Junit Tests - Summary

4 332 tests  +1   4 167 ✅ +2   11m 36s ⏱️ +53s
  456 suites  - 1     165 💤  - 1 
  456 files    - 1       0 ❌ ±0 

Results for commit da7d243. ± Comparison against base commit ac9d1f8.

♻️ This comment has been updated with latest results.

@gmagnu gmagnu merged commit 0c44072 into main Oct 24, 2025
11 checks passed
@gmagnu gmagnu deleted the ENGKNOW-2760-norsql-add-project-id-support-like-we-have-for-db-links branch October 24, 2025 15:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer

@bragnarsson bragnarsson bragnarsson approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@gmagnu @bragnarsson