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

core: sync BaseModel and FastCamelContext with Camel main #2750

Merged
merged 1 commit into from
Jun 9, 2021

Conversation

lburgazzoli
Copy link
Contributor

No description provided.

@lburgazzoli lburgazzoli requested review from davsclaus, jamesnetherton and ppalaga and removed request for davsclaus and jamesnetherton June 8, 2021 19:47
@lburgazzoli
Copy link
Contributor Author

@ppalaga @jamesnetherton more tests may be required, this is the bare minimum I got with the time I had today.
@davsclaus mind checking if I missed something ?

@davsclaus
Copy link
Contributor

Nice to see all this non functional code ;)

@lburgazzoli lburgazzoli merged commit c15189e into apache:main Jun 9, 2021
@lburgazzoli lburgazzoli deleted the camel-core-align branch June 9, 2021 08:35
@@ -1,4 +1,4 @@
<?xml version="1.0" encoding="UTF-8"?>
<?xml version="1.0" encoding="UTF-8"?>
Copy link
Contributor

Choose a reason for hiding this comment

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

Looks like you have added a byte order mark. Any reason for that?

$ hexdump -n 3 -C pom.xml 
00000000  ef bb bf                                          |...|

before the change:

$ hexdump -n 3 -C pom.xml 
00000000  3c 3f 78                                          |<?x|

@lburgazzoli
Copy link
Contributor Author

no reason

@ppalaga
Copy link
Contributor

ppalaga commented Jun 9, 2021

Thanks, let me revert the change

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.

6 participants