Skip to content

Comments

[Debug ]Removal of Deprecated Constants#1784

Merged
vogella merged 1 commit intoeclipse-platform:masterfrom
SougandhS:Deprecation_deletion
May 13, 2025
Merged

[Debug ]Removal of Deprecated Constants#1784
vogella merged 1 commit intoeclipse-platform:masterfrom
SougandhS:Deprecation_deletion

Conversation

@SougandhS
Copy link
Contributor

@SougandhS SougandhS commented Mar 19, 2025

This commit removes the deprecated constants in VariableColumnPresentation and updates references in DefaultVariableCellModifier and WatchExpressionCellModifier to use the new constants.

@github-actions
Copy link
Contributor

github-actions bot commented Mar 19, 2025

Test Results

 1 758 files  ±0   1 758 suites  ±0   1h 27m 38s ⏱️ + 2m 1s
 4 173 tests ±0   4 150 ✅ ±0   23 💤 ±0  0 ❌ ±0 
13 119 runs  ±0  12 952 ✅ ±0  167 💤 ±0  0 ❌ ±0 

Results for commit f828ae0. ± Comparison against base commit 02ffafb.

♻️ This comment has been updated with latest results.

@SougandhS
Copy link
Contributor Author

Hi @mickaelistria , could you please check this when you have time ?

@SougandhS SougandhS force-pushed the Deprecation_deletion branch from 18d4b04 to ad147ab Compare March 20, 2025 00:55
@SougandhS SougandhS force-pushed the Deprecation_deletion branch 2 times, most recently from 49f4443 to 91c5584 Compare May 7, 2025 01:06
This commit removes the deprecated constants in
VariableColumnPresentation and updates references in
DefaultVariableCellModifier and WatchExpressionCellModifier to use the
new constants.
@SougandhS SougandhS force-pushed the Deprecation_deletion branch from 91c5584 to f828ae0 Compare May 7, 2025 09:50
@SougandhS
Copy link
Contributor Author

Build is good now 👍

@vogella
Copy link
Contributor

vogella commented May 9, 2025

@jonahgraham the change is fine for you and CDT?

@jonahgraham
Copy link
Contributor

@jonahgraham the change is fine for you and CDT?

Yes

@vogella vogella merged commit 1cceead into eclipse-platform:master May 13, 2025
18 checks passed
@vogella
Copy link
Contributor

vogella commented May 13, 2025

Thanks @SougandhS

@SougandhS
Copy link
Contributor Author

Thank you @vogella & @iloveeclipse

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.

4 participants