quickfixj icon indicating copy to clipboard operation
quickfixj copied to clipboard

Assure compatibility with FIX Orchestra generated QFJ dictionary

Open david-gibbs-ig opened this issue 4 years ago • 7 comments

Fixes #336

Tests to assure compatibility with FIX Orchestra generated QFJ dictionary : issue-336

see https://github.com/quickfix-j/quickfixj/issues/336

david-gibbs-ig avatar Nov 10 '20 16:11 david-gibbs-ig

@chrjohn Is there a way to rerun failed checks , I'm pretty sure that concurrency problem in the JDK15 check is not due to this change :-)

david-gibbs-ig avatar Nov 10 '20 17:11 david-gibbs-ig

Can't you re-run the checks from the Checks tab? But maybe you are not allowed to do so. Otherwise close and reopen the PR should work but is of course ugly.

chrjohn avatar Nov 10 '20 18:11 chrjohn

git commit -m "retrigger checks" --allow-empty && git push <remote> HEAD I will try the close and re-open next time.

david-gibbs-ig avatar Nov 10 '20 18:11 david-gibbs-ig

@chrjohn I think that there are instabilities in the test cases. perhaps on windows specifically ? Do you see this concurrent modification exception in other builds ?

david-gibbs-ig avatar Nov 10 '20 19:11 david-gibbs-ig

The concurrent modification problems should get corrected by #324

chrjohn avatar Nov 11 '20 00:11 chrjohn

@david-gibbs-ig , could you please check the formatting on this change? It seems the indentation is sometimes two spaces, sometimes four... Thanks in advance.

chrjohn avatar Jan 17 '21 22:01 chrjohn

I've have converted the tabs to spaces, also updated Apache Mina version to address a security patch. See the release notes here https://mina.apache.org/mina-project/.

david-gibbs-ig avatar Dec 24 '21 13:12 david-gibbs-ig

This will be obsolete.

david-gibbs-ig avatar May 24 '23 12:05 david-gibbs-ig