Skip to content
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

Remove "Warning! Experimental Feature" on pull/merge? #1871

Open
tinchodias opened this issue Dec 9, 2024 · 5 comments
Open

Remove "Warning! Experimental Feature" on pull/merge? #1871

tinchodias opened this issue Dec 9, 2024 · 5 comments
Labels

Comments

@tinchodias
Copy link
Collaborator

Screenshot 2024-12-09 at 11 39 22

Even if there are issues for merge such as #1456, I feel we had this message too much time now and works pretty well. I use it maybe each week and I don't notice anything experimental.

What do you think about removing the warning? just merge?

@jecisc
Copy link
Contributor

jecisc commented Dec 9, 2024

I opened an issue for that a few years ago so I'd vote for yes!

@tinchodias
Copy link
Collaborator Author

Oh strange, I tried to find a related issue using the search bar but didn't have any result

@jecisc
Copy link
Contributor

jecisc commented Dec 9, 2024

#1155

@tinchodias
Copy link
Collaborator Author

Ah, so this issue is duplicated. Should we close it?

@tinchodias
Copy link
Collaborator Author

I see @guillep was against removing the warning.

I think it's a good practice to save the image before doing the merge, to recover. Anyway, Epicea records the Iceberg change... but sometimes the change can be hidden if the loaded code is large.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants